Update k8s.io/cri-api to v0.25.0-alpha2#7114
Conversation
|
Hi @rata. Thanks for your PR. I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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. |
|
Run |
|
Hmm, will we have a release with the alpha version of kubernetes in our go.mod? (i'd rather not!) |
|
Uhm, CI sesm to have issues connecting to some mirrors: will hard push with no changes to restart the job |
|
@dims why not? Can you please elaborate? We will change it before the containerd final release, of course. We did it already in b69bbe2, for example. I don't see what it can be the problem. The diff is quite simple (https://github.com/containerd/containerd/pull/7114/files#diff-46912defa88ac8ecb8a26d0bb63c852b326099ab033b0385fa4d670776f39f69) and was reviewed several times in the KEP and in the code change. Sure, we will change the version used in go.mod to the final one later (once that is available), but it will not change the code it will just change the name of the tag |
3206aa8 to
b5effdd
Compare
This version contains the CRI changes for user namespaces support. Future patches will use the new fields in the CRI. Updating the module without using the new fields doesn't cause any behaviour change. Updates: containerd#7063 Signed-off-by: Rodrigo Campos <[email protected]>
|
Rebased and the CI is happy now :) |
What is the timeline for 0.25? It seems we have enough time before the release to get this off alpha |
|
@dmcgowan The final release is expected by end of august (https://github.com/kubernetes/sig-release/tree/master/releases/release-1.25), so yes, we should have lot of time to update before the final containerd release. |
|
/ok-to-test |
This version contains the CRI changes for user namespaces support.
Future patches will use the new fields in the CRI.
Updating the module without using the new fields doesn't cause any
behaviour change.
Updates: #7063
Signed-off-by: Rodrigo Campos [email protected]