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

feat(t8s-cluster/management-cluster): validate new k8s version before upgrade #508

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

cwrau
Copy link
Member

@cwrau cwrau commented Aug 23, 2023

Currently the user can (accidentally) downgrade the k8s version, which is of course not supported

This job checks the new version against the running version and fails it is lower

@cwrau cwrau requested a review from a user August 23, 2023 08:57
@cwrau cwrau enabled auto-merge (squash) August 23, 2023 08:58
@teutonet-bot
Copy link
Contributor

🤖 I have diffed this beep boop

"/$namespace/$kind/$name.yaml" for normal resources
"/$namespace/HelmRelease/$name/$namespace/$kind/$name.yaml" for HelmReleases <- this is recursive
'null' means it's either cluster-scoped or it's in the default namespace for the HelmRelease

charts/t8s-cluster/ci/artifacthub-values.yaml

charts/t8s-cluster/ci/cidrs-values.yaml

charts/t8s-cluster/ci/injectedCertificateAuthorities-values.yaml

charts/t8s-cluster/ci/securityGroups-values.yaml

charts/t8s-cluster/ci/calico-values.yaml has no changes

charts/t8s-cluster/ci/bastion-values.yaml

charts/t8s-cluster/values.yaml

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