Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[receiver/vcenter] Fixes vcenter.vm.disk.throughput metric #32780

Conversation

StefanKurek
Copy link
Contributor

Description:
Adds missing functionality for vcenter.vm.disk.throughput metric on VM resources.
Also changes the metric from a Sum to a Gauge.

Link to tracking Issue:
#32772

Testing:
Unit/integration tests updated and tested. Local environment tested.

Documentation:
New documentation generated based on the metadata.

@StefanKurek StefanKurek force-pushed the fix/vcenterreceiver_add_vm_disk_throughput branch from cc32773 to dfb7b6f Compare April 30, 2024 19:55
@djaglowski djaglowski merged commit 9c96d14 into open-telemetry:main May 1, 2024
170 checks passed
@github-actions github-actions bot added this to the next release milestone May 1, 2024
@StefanKurek StefanKurek deleted the fix/vcenterreceiver_add_vm_disk_throughput branch May 1, 2024 18:56
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
…lemetry#32780)

**Description:** <Describe what has changed.>
Adds missing functionality for `vcenter.vm.disk.throughput` metric on VM
resources.
Also changes the metric from a Sum to a Gauge.

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#32772 

**Testing:** <Describe what testing was performed and which tests were
added.>
Unit/integration tests updated and tested. Local environment tested.

**Documentation:** <Describe the documentation added.>
New documentation generated based on the metadata.
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
…lemetry#32780)

**Description:** <Describe what has changed.>
Adds missing functionality for `vcenter.vm.disk.throughput` metric on VM
resources.
Also changes the metric from a Sum to a Gauge.

**Link to tracking Issue:** <Issue number if applicable>
open-telemetry#32772 

**Testing:** <Describe what testing was performed and which tests were
added.>
Unit/integration tests updated and tested. Local environment tested.

**Documentation:** <Describe the documentation added.>
New documentation generated based on the metadata.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants