-
Notifications
You must be signed in to change notification settings - Fork 18
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
bump go to 1.22 #188
base: main
Are you sure you want to change the base?
bump go to 1.22 #188
Conversation
obsoletes #190 |
I will squash this PR into a single commit because none of commits in the PR work without all the other commits in the PR. |
@mcbenjemaa you need to be more specific - in what way are they not working? |
sorry, my caches |
We've decided against merging this until a cluster-api release with controller-runtime v0.18.0 support comes out. |
None of the providers moved to 1.22
I recommend that we wait for the v1.8 release. To be compliant with CAPI. |
capi 1.7.2 still does not support controller-runtime 0.18 |
Plus relevant dependency updates. fix #187
Rebased for v0.5.0. |
Quality Gate passedIssues Measures |
Issue #, if available:
fix #187
+mass dep update:
k8s/api* to 0.30
controller-gen to 0.15.0 (0.14.0 has the go 1.22 panic fix)
controller-runtime to 0.18.0 (dependency)
Regenerated crds using new controller-gen. Note config/rbac/role.yaml and config/webhook/manifest.yaml.