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

Rationalize Versions #14

Merged
merged 1 commit into from
Jul 19, 2024
Merged

Rationalize Versions #14

merged 1 commit into from
Jul 19, 2024

Conversation

spjmurray
Copy link
Member

We had to duplicate the Kubernetes version in multiple places, which makes configuration harder, and is a classic case of over-engineering when any iupgrade logic should already be part of CAPI. I've bumped this to a new major alpha version too as legacy naming is now dead, and we shouldn't be supporting something that is inherently broken because it will just waste time.

We had to duplicate the Kubernetes version in multiple places, which
makes configuration harder, and is a classic case of over-engineering
when any iupgrade logic should already be part of CAPI.  I've bumped
this to a new major alpha version too as legacy naming is now dead, and
we shouldn't be supporting something that is inherently broken because
it will just waste time.
@spjmurray
Copy link
Member Author

Tested against a live cluster, nothing untoward happened during the upgrade (after the bug fixes!) so high confidence this is good!

@spjmurray spjmurray merged commit b71526f into main Jul 19, 2024
1 check passed
@spjmurray spjmurray deleted the cleanups branch July 19, 2024 08:51
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.

1 participant