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

Add kubeVersion to Chart.yaml to enforce minimum Kubernetes version #443

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jan 24, 2024

k8s 1.21 is the minimum required version

Copy link
Contributor

@frankjkelly frankjkelly left a comment

Choose a reason for hiding this comment

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

Nice LGTM!

@lhotari lhotari merged commit de4d2e7 into apache:master Jan 24, 2024
25 of 27 checks passed
@frankjkelly
Copy link
Contributor

FYI @lhotari I noticed that the CI test Upgrade latest released version - k8s 1.27.3 - upgrade failed - is that expected?

@lhotari
Copy link
Member Author

lhotari commented Jan 24, 2024

FYI @lhotari I noticed that the CI test Upgrade latest released version - k8s 1.27.3 - upgrade failed - is that expected?

I'm re-running the failed jobs to see if it was a flakiness issue.

@frankjkelly
Copy link
Contributor

frankjkelly commented Jan 24, 2024

FYI @lhotari I noticed that the CI test Upgrade latest released version - k8s 1.27.3 - upgrade failed - is that expected?

I'm re-running the failed jobs to see if it was a flakiness issue.

Sounds good - should the repo have branch protection settings to prevent a merge in such a case (barring an explicit override)?

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.

2 participants