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

vcenterreceiver: Adds vcenter.datacenter.name attributes to all resource types #32528

Merged

Conversation

StefanKurek
Copy link
Contributor

@StefanKurek StefanKurek commented Apr 18, 2024

Description:
Adds vcenter.datacenter.name attribute for every resource type. This ensures uniqueness in cases where more than one datacenter exists.

Link to tracking Issue:
#32531

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

Documentation:
Documentation regenerated.

@StefanKurek StefanKurek requested a review from djaglowski as a code owner April 18, 2024 18:42
@StefanKurek StefanKurek requested a review from a team April 18, 2024 18:42
@github-actions github-actions bot requested a review from schmikei April 18, 2024 18:43
@StefanKurek StefanKurek force-pushed the feat/vcenterreceiver-datacenter-attrs branch 5 times, most recently from 5f23064 to 88cef28 Compare April 18, 2024 20:49
@github-actions github-actions bot added the cmd/configschema configschema command label Apr 18, 2024
@StefanKurek StefanKurek force-pushed the feat/vcenterreceiver-datacenter-attrs branch 2 times, most recently from da505e8 to c71b871 Compare April 18, 2024 21:12
@github-actions github-actions bot added the cmd/otelcontribcol otelcontribcol command label Apr 18, 2024
@StefanKurek StefanKurek force-pushed the feat/vcenterreceiver-datacenter-attrs branch from b2bbfc9 to bd96792 Compare April 19, 2024 17:20
@StefanKurek StefanKurek requested a review from djaglowski April 19, 2024 17:55
@StefanKurek StefanKurek force-pushed the feat/vcenterreceiver-datacenter-attrs branch 3 times, most recently from fb970ad to 5bb5089 Compare April 23, 2024 21:03
@StefanKurek
Copy link
Contributor Author

@djaglowski should be ready for your review now. Also added the warning in the metadata for the new resource attribute.

receiver/vcenterreceiver/metadata.yaml Outdated Show resolved Hide resolved
StefanKurek and others added 4 commits April 24, 2024 11:35
vcenterreceiver `vcenter.datacenter.name` resource attribute is disabled by default.
adds warning message that it will be enabled starting in release v.0.100.0.
@StefanKurek StefanKurek force-pushed the feat/vcenterreceiver-datacenter-attrs branch from 9d06538 to 53e4669 Compare April 24, 2024 15:43
@StefanKurek
Copy link
Contributor Author

@djaglowski should have all conflicts fixed now.

@djaglowski djaglowski merged commit 3c64cea into open-telemetry:main Apr 24, 2024
170 of 171 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 24, 2024
@StefanKurek StefanKurek deleted the feat/vcenterreceiver-datacenter-attrs branch April 24, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd/configschema configschema command cmd/otelcontribcol otelcontribcol command receiver/vcenter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants