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
Automated cherry pick of #114770: Fix clearing rate limiter in disruption controller #114780
Automated cherry pick of #114770: Fix clearing rate limiter in disruption controller #114780
Conversation
/assign @alculquicondor |
/assign @soltysh |
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
/approve
LGTM label has been added. Git tree hash: e658349b2bf017ec1a0b703041d401af10d204aa
|
/kind regression |
/cc kubernetes/release-managers |
It also affects 1.25, @xmudrii should I prepare an analogous PR? |
@mimowo Yes, please. |
Actually, I'm in a doubt after a second thought. The code wasn't protected by a feature gate in 1.25, however, it was indirectly protected as it would only activate if the pod disruption condition was added (https://github.com/kubernetes/kubernetes/blob/release-1.25/pkg/controller/disruption/disruption.go#L487-L489), and adding pod disruption conditions was alpha in 1.25. @alculquicondor @soltysh WDYT? Closing the other PR for 1.25 for now. |
Based on general guidelines for cherry-picks 1.25 wouldn't have to be fixed in this case. So I'm inclined to say to not to fix it there. |
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mimowo, soltysh, xmudrii 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 |
Cherry pick of #114770 on release-1.26.
#114770: Fix clearing rate limiter in disruption controller
For details on the cherry pick process, see the cherry pick requests page.