-
Notifications
You must be signed in to change notification settings - Fork 502
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
Revert bumping kubernetes-cni to v1.2.0 and cri-tools to v1.26.0 #2875
Conversation
This reverts commit 024e778.
This reverts commit f1da4c4.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Thank you for this fix! Are there plans to remove the kubernetes-cni v1.2.0 and cri-tools v1.26.0 packages from the repos? Currently I still see it (atleast in the Deb repo) which means when |
We don't have such plans as of now and I'm also not sure if it is even technically feasible to remove packages. There are a few important notes regarding this PR:
|
What type of PR is this?
/kind regression
What this PR does / why we need it:
We bumped the minimum kubernetes-cni version to v1.2.0 in #2863, and cri-tools to v1.26.0 in #2821. However, this affects all kubelet and kubeadm packages, including v1.23.16, v1.24.10, v1.25.6, and v1.26.1.
This brings two major problems:
Therefore, we think it might be better to revert those bumps. This problem will be solved in the future with the KEP-1731 (kubernetes/enhancements#1731).
Which issue(s) this PR fixes:
Fixes #2866
Does this PR introduce a user-facing change?
/assign @saschagrunert
cc @kubernetes/release-engineering