-
Notifications
You must be signed in to change notification settings - Fork 373
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 Kubernetes to v1.31.0 #4647
Conversation
8f0ce6c
to
b4e2473
Compare
670d04d
to
65e991c
Compare
This pull request has merge conflicts that need to be resolved. |
65e991c
to
f14ea50
Compare
dee9717
to
3e42b15
Compare
9d56c51
to
eded00b
Compare
This pull request has merge conflicts that need to be resolved. |
eded00b
to
c54c5f2
Compare
49c146e
to
ab4326a
Compare
This pull request has merge conflicts that need to be resolved. |
ab4326a
to
d53166b
Compare
This pull request has merge conflicts that need to be resolved. |
d53166b
to
b454134
Compare
2bfd6d9
to
d830f04
Compare
This pull request has merge conflicts that need to be resolved. |
d830f04
to
52059fa
Compare
1c6b013
to
8a7056a
Compare
This pull request has merge conflicts that need to be resolved. |
8a7056a
to
836a5cf
Compare
This pull request has merge conflicts that need to be resolved. |
Allows for reusing the internalize error handling. Signed-off-by: Tom Wieczorek <[email protected]>
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.31.md#changelog-since-v1300 * Adapt fake client's ObjectTracker to k/k PR 125560 The interface changed upstream, add the variadic option arguments and missing Patch and apply methods. Moreover, remove the capacity from the external channel for Watch. This could introduce double-buffering. * Bump controller-runtime to v0.19.0 https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.19.0 This is the version which is compatible with Kubernetes 1.31. * Skip name checks when restarting Autpoilot root controller If the autpoilot root controller is started multiple times due to reconfiguration events, this means that all controller-runtime controllers have already been successfully registered to another manager during the initial startup. However, controller-runtime maintains a global checklist of controller names and doesn't currently provide a way to unregister names from discarded managers. So it's necessary to suppress the global name check each time things are restarted for reconfiguration. Signed-off-by: Tom Wieczorek <[email protected]>
So that the number of matched packages can be asserted separately, if required. Signed-off-by: Tom Wieczorek <[email protected]>
The Kubernetes 1.31 bump reintroduced the transitive runc go dependency again. Restore the module version check for it. See: 537ebdb ("Bump containerd to v1.7.12") Signed-off-by: Tom Wieczorek <[email protected]>
836a5cf
to
dfaed15
Compare
dfaed15
to
5d7b4b4
Compare
Description
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.31.md#changelog-since-v1300
Restore runc module version test
The Kubernetes 1.31 bump reintroduced the transitive runc go dependency again. Restore the module version check for it. See Bump containerd to v1.7.12 #3928.
Adapt fake client's ObjectTracker to k/k PR 125560
The interface changed upstream, add the variadic option arguments and missing Patch and apply methods. Moreover, remove the capacity from the external channel for Watch. This could introduce double-buffering. Introduce internalized helper, which allows for reusing the internalize error handling.
Bump controller-runtime to v0.19.0
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.19.0
This is the version which is compatible with Kubernetes 1.31.
Skip name checks when restarting Autpoilot root controller
If the autpoilot root controller is started multiple times due to reconfiguration events, this means that all controller-runtime controllers have already been successfully registered to another manager during the initial startup. However, controller-runtime maintains a global checklist of controller names and doesn't currently provide a way to unregister names from discarded managers. So it's necessary to suppress the global name check each time things are restarted for reconfiguration.
See:
Type of change
How Has This Been Tested?
Checklist: