-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Promote WatchBookmarks feature to GA #83195
Conversation
98b70f6
to
512c0a9
Compare
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
I'm in favor of this going to GA. The reduced load on etcd is certainly welcome. |
@jpbetz - I don't think the impact on etcd is actually very significant (if any). We're serving watches from apiserver anyway. So it help apiserver (and watchcache contention). |
512c0a9
to
1e8a96c
Compare
@@ -161,7 +162,7 @@ var defaultKubernetesFeatureGates = map[featuregate.Feature]featuregate.FeatureS | |||
StorageVersionHash: {Default: true, PreRelease: featuregate.Beta}, | |||
WinOverlay: {Default: false, PreRelease: featuregate.Alpha}, | |||
WinDSR: {Default: false, PreRelease: featuregate.Alpha}, | |||
WatchBookmark: {Default: true, PreRelease: featuregate.Beta}, | |||
WatchBookmark: {Default: true, PreRelease: featuregate.GA, LockToDefault: true}, // remove in 1.21 |
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.
Remove what in 1.21? We shouldn't need to wait that long before removing the feature gate itself?
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.
Removed
/approve Anyone can lgtm once my question is answered :) |
44ec004
to
9508d13
Compare
9508d13
to
ef38f6d
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lavalamp, wojtek-t The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@lavalamp - PTAL |
/lgtm |
Ref kubernetes/enhancements#956