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

[chore] Remove configuration for disabling opencensus.resourcetype #963

Merged

Conversation

dmitryax
Copy link
Contributor

@dmitryax dmitryax commented Oct 13, 2023

The resource attribute was disabled upstream in 0.87.0 and will be removed in the next version.

This change has no effect on the end users, so no need to mention in the changelog

@dmitryax dmitryax requested review from a team as code owners October 13, 2023 06:12
@atoulme
Copy link
Contributor

atoulme commented Oct 13, 2023

The tests fail because we need to update the collector to 0.87.0 to remove this attribute.

@dmitryax dmitryax force-pushed the remove-opencensus.resourcetype-from-config branch from ce78768 to 9033490 Compare October 14, 2023 02:30
@dmitryax dmitryax changed the title Remove configuration for disabling opencensus.resourcetype [chore] Remove configuration for disabling opencensus.resourcetype Oct 14, 2023
@dmitryax
Copy link
Contributor Author

The tests fail because we need to update the collector to 0.87.0 to remove this attribute.

You're right. I confused the version. Turning it to draft until 0.87.0 is released

@dmitryax dmitryax marked this pull request as draft October 14, 2023 03:19
@dmitryax
Copy link
Contributor Author

This should be merged along with the 0.87.0 release or after 0.87.0 image adoption and before the release. Otherwise, there will be some deprecation messages thrown

The resource attribute was disabled upstream in 0.86.0 and will be removed in the next version.

This change has no effect on the end users, so no need to mention in the changelog
@dmitryax dmitryax force-pushed the remove-opencensus.resourcetype-from-config branch from 9033490 to 698b63d Compare November 1, 2023 04:22
Copy link
Contributor

github-actions bot commented Nov 1, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@dmitryax dmitryax marked this pull request as ready for review November 1, 2023 04:22
@dmitryax
Copy link
Contributor Author

dmitryax commented Nov 1, 2023

@atoulme, rebased. PTAL

@dmitryax
Copy link
Contributor Author

dmitryax commented Nov 1, 2023

I have read the CLA Document and I hereby sign the CLA

srv-gh-o11y-gdi-cla added a commit to splunk/cla-agreement that referenced this pull request Nov 1, 2023
@dmitryax dmitryax merged commit 1c06636 into signalfx:main Nov 1, 2023
25 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants