-
Notifications
You must be signed in to change notification settings - Fork 220
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
added pdb version detection #260
added pdb version detection #260
Conversation
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.
should be backward compatible, lgtm
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.
I think we'll need to take this comment into consideration here #183 (comment). @yuweisung are you able to test if this is an issue for this PR? Thanks!
update the capabilities syntax
update capability syntax
update capability version syntax
update capability version syntax
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
Thanks @yuweisung! |
fix typo
I verified manually that this PR works against minikube running k8s v1.25.2. |
Fixes pod disruption budget version warning
Motivation
PDB policy api version, v1beta1 is deprecated in k8s1.21+ (not available in 1.25+).
Modifications
zookeeper-pdb, proxy-pdb, broker-pdb and bookkeepr-pdb templates are modified. If k8s api-resources container policy/v1, the *-pdb.yaml will generate respective apiVersion.
Verifying this change