-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 telemetry.useOtelForInternalMetrics
stable feature gate
#9752
[chore] Remove telemetry.useOtelForInternalMetrics
stable feature gate
#9752
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9752 +/- ##
==========================================
- Coverage 90.99% 90.97% -0.02%
==========================================
Files 353 353
Lines 18614 18614
==========================================
- Hits 16937 16934 -3
- Misses 1352 1354 +2
- Partials 325 326 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @codeboten
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @codeboten
Not sure how I approved twice, lol |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should have a changelog with breaking change right? as users will no longer be able to specify the feature gate
I wasn't sure. I looked at previous removal of stable feature gate and it had no changelog entry. |
Right, i think it should always be in the changelog, otherwise the collector won't start with:
|
Since the gate is already stable removing it is a non-breaking change. Since it is stable already, the collector will error on startup if any user attempts to set the gate to any value, so at this point all users should have removed the gate from their startup flags or the collector wont start. |
@TylerHelmuth right, but the collector would continue to start until this change goes out no? isn't that what makes it a breaking change? |
Well I didnt think it did but I was wrong. So ya, we need a changelog. |
I'll add a changelog, one moment. |
56f6b12
to
f34a218
Compare
Description:
Remove the stable feature gate
telemetry.useOtelForInternalMetrics
. It was declared stable in 0.95.0.