-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Upgrade Kubernetes API to 1.27 #82
Conversation
Maybe we could also bump CI acceptance test Kube version against v1.27.2 |
Here 😉 #83 (with 1.27.3 though) |
These are breaking change, right?
Do we need a webhook and a new API version to support backward compatibility (ie. if someone updates the operator to a version with this change, it shouldn't break existing instances)? |
Please rebase instead of merging main. |
5966a56
to
58b1797
Compare
Signed-off-by: Ramiz Polic <ramiz.polic@hotmail.com>
Signed-off-by: Ramiz Polic <ramiz.polic@hotmail.com>
58b1797
to
2e00e1f
Compare
I will move API these changes to a separate PR since it will require a bit more work. This PR does not really benefit from those changes, so I will rollback to ones without it. |
Signed-off-by: Ramiz Polic <ramiz.polic@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Overview
Tracks #47
Notes for reviewer
controller-runtime
interfaces have slightly changed, the new changes take those into account