-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update docker 20.10.6 #3554
Update docker 20.10.6 #3554
Conversation
a8cc06b
to
2c07ef0
Compare
The Is containerd 1.4.4 required for docker 20.10.6? Or can it run on containerd 1.4.1? If it is not critically required, I would recommend taking out the containerd change, push the others through, and then we can do a separate PR just for that. If it is definitely required, then let's hold off on this until the other PR #3623 is in, and we can do a separate (but much smaller) alpine one. |
Ah thanks! Yes, that's the part I wasn't sure about
I think technically 1.3.x would work as well, but we specify v1.4.1 as minimum version, as that's what we shipped/tested Docker 20.10.0 with; https://github.com/docker/docker-ce-packaging/blob/b30ac69f11d8046e41478c9e901a66894690a320/deb/common/control#L30 There were some bug fixes in patch releases after v1.4.1 though, so would be good to have.
Sounds ok to me; I just realised that I also forgot to update runc, which looks to be handled separately; Line 14 in 4542998
About runc; let me add some info (for when you'll be working on updating containerd); If you are updating containerd, and want to know the "matching" runc version (that containerd has been tested against); upcoming releases will have the version specified in a separate file; see this pull request for more details: containerd/containerd#5036 Docker currently ships runc version v1.0.0-rc93 because it fixes some issues that were commonly hit, however depending on how it's used, you may want to use v1.0.0-rc92 or (hopefully soon..) v1.0.0-rc94, because while rc92 fixed some issues, it also introduced a regression in other regions; see containerd/containerd#5289 and opencontainers/runc#2865) - yeah we're a bit between a rock and a hard place until rc94 is released 😞 |
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2c07ef0
to
6c27d84
Compare
Reverted the containerd update 👍 |
Sounds like we should get this in (just did "LGTM"), and then, when #3623 is done and merged, do a separate update for containerd 1.4.4, and one for runc. Isn't containerd 5.x supposed to be on the cusp? |
Yes, containerd 1.5.0 is getting close, but containerd maintains a 6 month support overlap between releases, so version 1.4 will continue to be maintained until (at least) 6 months after v1.5.0; https://github.com/containerd/containerd/blob/master/RELEASES.md#support-horizon Docker will likely switch to 1.5 with the docker 21.xx release (date TBD); we have done some tests in CI to verify containerd doesn't break there, but we're good at "finding the odd bugs" once people start using it en-masse, so perhaps using "latest 1.4.x" as a stepping-stone between would be good |
Ping? Maybe also include latest containerd 1.5.2? PS: I'm looking forward for using cgroup v2! |
Probably should be updated to containerd 1.4.6 and runc v1.0.0-rc95 |
As suggested on linuxkit#3554 (comment) Signed-off-by: David Scott <dave@recoil.org>
@thaJeztah I've made separate PRs for containerd.1.4.6 and runc-v1.0.0-rc95 as you suggested! |
As suggested on linuxkit#3554 (comment) Signed-off-by: David Scott <dave@recoil.org>
As suggested on linuxkit#3554 (comment) Signed-off-by: David Scott <dave@recoil.org>
As suggested on linuxkit#3554 (comment) Signed-off-by: David Scott <dave@recoil.org>
Wasn't sure if anything else was needed to bring these up to date 😅
/cc @rn @justincormack