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

Update PodDisruptionBudgets API version #835

Merged
merged 4 commits into from
Jun 27, 2023
Merged

Conversation

legoGoat
Copy link
Contributor

@legoGoat legoGoat commented Jun 27, 2023

Fixes #833 PodDisruptionBudget policies fail to create at k8s v1.25+

Description: Capabilities.KubeVersion.Version checks the kube version. If above 1.21, apiVersion: policy/v1 is set for PodDisruptionBudget, else apiVersion: policy/v1beta1.

@legoGoat legoGoat requested review from a team as code owners June 27, 2023 19:52
@legoGoat legoGoat changed the title added version v1 as an option added version policy/v1 as an option Jun 27, 2023
Copy link
Contributor

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you, @legoGoat!

@dmitryax dmitryax changed the title added version policy/v1 as an option Update PodDisruptionBudgets API version Jun 27, 2023
@legoGoat
Copy link
Contributor Author

@dmitryax hi, sorry, any advice on the test that failed? It was the functional tests bit that failed and I wouldn't think this change would affect any changes... Could this be fixed just rerunning the workflow or is there something obvious I'm missing that's at fault?

@dmitryax
Copy link
Contributor

That test is flaky, sorry

@legoGoat
Copy link
Contributor Author

Not a problem, just got me worried there for a second 🙈
Now if it succeeds this time around, should I dare to push an extra commit with a CHANGELOG.md change?

@dmitryax
Copy link
Contributor

Now if it succeeds this time around, should I dare to push an extra commit with a CHANGELOG.md change?

Sure, thank you!

@legoGoat
Copy link
Contributor Author

oh no, functional tests strike again 😢 though it's cri-o this time, not docker

@atoulme
Copy link
Contributor

atoulme commented Jun 27, 2023

rerunning.

@dmitryax dmitryax merged commit 06d3588 into signalfx:main Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PodDisruptionBudget policies fail to create at k8s v1.25+
3 participants