Skip to content
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

Fix policy version #8600

Merged
merged 2 commits into from
Feb 6, 2023
Merged

Conversation

vrazdalovschi
Copy link
Contributor

Fixes #7222

After bumping in #8313 not all places were updated to the new version. It causes issues on k8s 1.25+, where v1beta is not working anymore. This PR fixes skipped API version and remove comments related to the previous beta version.

Backports Required

  • none - not a bug fix
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v22.3.x
  • v22.2.x
  • v22.1.x

UX Changes

None

Release Notes

Bug Fixes

  • Pod distribution budget version v1 is used now in the operator

Improvements

  • Cleaned up comments.

`v1beta1` was upgraded to `v1`, but not changed in all places. Fix skipped part.
after bumping from `v1beta1` to `v1` comment is not relevant.
@vrazdalovschi vrazdalovschi requested a review from a team as a code owner February 3, 2023 11:33
@CLAassistant
Copy link

CLAassistant commented Feb 3, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@RafalKorepta RafalKorepta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution

@RafalKorepta RafalKorepta merged commit f10f8f2 into redpanda-data:dev Feb 6, 2023
@vrazdalovschi vrazdalovschi deleted the fix_policy_version branch February 6, 2023 10:59
@vrazdalovschi
Copy link
Contributor Author

@RafalKorepta, How could we backport #8313 and this one PR to 22.3.x? Right now, I manually build the docker image and use it, but I'd like to switch back to the redpanda images as soon as it is fixed in 22.3.x version.

@vshtokman
Copy link
Contributor

/backport v22.3.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

operator not working with k8s 1.25+ due to policy/v1beta1 PodDisruptionBudget
4 participants