-
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
Mark connectors as stable #7539
Conversation
d2b0323
to
638c74d
Compare
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.
LGTM, will need to set a version for when the gate will be removed. See
panic: no removal version set for stable gate "service.connectors"
a74232e
to
b32a027
Compare
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #7539 +/- ##
=======================================
Coverage 91.18% 91.18%
=======================================
Files 296 296
Lines 14422 14422
=======================================
Hits 13151 13151
Misses 1006 1006
Partials 265 265 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Should be good to go now. |
@@ -19,6 +19,7 @@ import "go.opentelemetry.io/collector/featuregate" | |||
|
|||
var ConnectorsFeatureGate = featuregate.GlobalRegistry().MustRegister( | |||
"service.connectors", | |||
featuregate.StageBeta, | |||
featuregate.StageStable, |
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.
Add please the "FromVersion"
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.
Added, backdated to v0.71.0.
Can we close #2336 prior to merging this PR? |
b32a027
to
004dff3
Compare
@atoulme, done. |
I believe the connectors codebase has proven to be very stable, as we haven't had to make any changes to support development of individual connectors. If there are no objections, I'd like to mark the gate stable ahead of the next release.