-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 Kustomize to v5 #8457
Comments
Let's consider if to wait for a patch release with the fix for creationTimestamp |
@fabriziopandini: GuidelinesPlease ensure that the issue body includes answers to the following questions:
For more details on the requirements of such an issue, please see here and ensure that they are met. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Yeah - I'm watching that issue to see if there's an update. We should consider it a blocker until then IMO. |
Should still be able to work through the deprecations in the mean-time. |
I've opened a related issue to bump controller-tools: #8546 As far as I can tell we should be able to bump kustomize after we bumped controller-tools? |
/kind cleanup |
/priority important-soon |
Done in #10294 /close @killianmuldoon Maybe you can double check. But as far as I can telle verything is done (and we now use kustomize v5.3) |
@sbueringer: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Kustomize released v5.0.0 in February - CAPI should bump the Kustomize version to keep up. Release notes are here.
The following need to be taken care of ahead of and / or alongside the Kustomize bump:
creationTimestamp
is handled correctly. This should be handled in upstream Kustomize - issue here - but if not there are workarounds for CAPI - e.g. patchcreationTimestamp
on each and every CRD. Reference issue in CAPI./area dependency
The text was updated successfully, but these errors were encountered: