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] Updates vcenter.cluster.memory.used Metric Config With Warning #32807

Conversation

StefanKurek
Copy link
Contributor

Description:
This metric is not actually implemented. Adding a warning for now if a user has it configured in some way to let them know that it'll be going away soon.

Link to tracking Issue:
#32805

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

Documentation:
N/A

@djaglowski djaglowski merged commit a472e30 into open-telemetry:main May 2, 2024
178 checks passed
@github-actions github-actions bot added this to the next release milestone May 2, 2024
@StefanKurek StefanKurek deleted the fix/vcenterreceiver-cluster-metric-warn branch May 2, 2024 15:46
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
…g With Warning (open-telemetry#32807)

**Description:** <Describe what has changed.>
This metric is not actually implemented. Adding a warning for now if a
user has it configured in some way to let them know that it'll be going
away soon.

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

**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.>
N/A
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
…g With Warning (open-telemetry#32807)

**Description:** <Describe what has changed.>
This metric is not actually implemented. Adding a warning for now if a
user has it configured in some way to let them know that it'll be going
away soon.

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

**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.>
N/A
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