Fix PDB by percentages for StatefulSet pods#39454
Conversation
|
Jenkins Bazel Build failed for commit 1508ecf. Full PR test history. The magic incantation to run this job again is DetailsIf you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
Jenkins verification failed for commit 1508ecf. Full PR test history. The magic incantation to run this job again is DetailsIf you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
Thanks @foxish! |
|
LGTM |
|
Automatic merge from submit-queue (batch tested with PRs 39435, 39454) |
|
@saad-ali, can we cherrypick this fix into the next 1.5.x release? |
|
Removing label |
|
If this is cherrypicked, so should #39544 |
…39544-upstream-release-1.5 Automatic merge from submit-queue Automated cherry pick of #39454 #39544 Cherry pick of #39454 #39544 on release-1.5. #39454: Make PDBs represent percentage in StatefulSet #39544: Allow disruption controller to read statefulsets /cc @davidopp @maisem @kubernetes/sig-apps-misc **Release note**: ``` Fixed bug which prevents PDBs which have minAvailable specified as a percentage working with StatefulSet pods ```
|
Commit found in the "release-1.5" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked. |
Previously, PDBs defined in terms of percentages would error out with StatefulSet as they did not know how to find the scale associated.
This change teaches the disruption controller to also look at StatefulSets and their scale.
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged): fixes #39125Release note:
cc @a-robinson @kow3ns @kubernetes/sig-apps-misc