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 kubebuilder #177

Merged
merged 7 commits into from
Oct 17, 2024
Merged

Update kubebuilder #177

merged 7 commits into from
Oct 17, 2024

Conversation

jabdoa2
Copy link
Contributor

@jabdoa2 jabdoa2 commented Oct 16, 2024

  • Bump controller-runtime version to 0.19.0
  • Bump kubernetes api version to 1.31
  • Update golang to 1.23
  • Update to Alpine 3.20
  • Update golang-ci to 1.61.0

Unfortunately, we currently cannot upgrade further to controller-runtime 0.19 due to issues with kubernetes 1.31 api and ginko v2. See kubernetes/code-generator#177 for details. This should land in controller-runtime soonish.

@jabdoa2 jabdoa2 requested a review from a team as a code owner October 16, 2024 09:43
@toelke toelke merged commit 43372fa into wave-k8s:master Oct 17, 2024
6 checks passed
@toelke
Copy link
Collaborator

toelke commented Oct 17, 2024

Thank you for the contribution!

@jabdoa2
Copy link
Contributor Author

jabdoa2 commented Oct 17, 2024

Can you tag a release in the next few days? We learned the hard way that using a 1.29 client on kubernetes 1.30 can still cause issues. By definition the api client is compatible for +/- one kubernetes version. However, we did not know that this does not apply to any alpha APIs. In this case the mutating webhook will simply remove those newer APIs in pods which can cause adverse side-effects in some charts.

@toelke
Copy link
Collaborator

toelke commented Oct 18, 2024

Done, chart version 4.4.0.

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