Skip to content
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

Merged
merged 1 commit into from
Jun 2, 2021
Merged

Conversation

thaJeztah
Copy link
Contributor

Wasn't sure if anything else was needed to bring these up to date 😅

/cc @rn @justincormack

@thaJeztah thaJeztah force-pushed the update_docker_containerd branch from a8cc06b to 2c07ef0 Compare April 21, 2021 11:18
@thaJeztah thaJeztah changed the title Update docker 19.03.13, containerd v1.3.7 Update docker 20.10.6, containerd 1.4.4 Apr 21, 2021
@thaJeztah
Copy link
Contributor Author

Updated to docker 20.10.6 and the latest containerd v1.4.x release

@rn @djs55 @deitch ptal (hope I did this right)

@deitch
Copy link
Collaborator

deitch commented Apr 21, 2021

The examples/ and test/ all look fine. The change to tools/alpine is a lot more convoluted than that. I am in the process of updating everything (and I mean, everything) with an update to tools/alpine. It has been a few days of work.

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.

@thaJeztah
Copy link
Contributor Author

thaJeztah commented Apr 22, 2021

The change to tools/alpine is a lot more convoluted than that.

Ah thanks! Yes, that's the part I wasn't sure about

Is containerd 1.4.4 required for docker 20.10.6? Or can it run on containerd 1.4.1?

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.

let's hold off on this until the other PR #3623 is in, and we can do a separate (but much smaller) alpine one.

Sounds ok to me; I just realised that I also forgot to update runc, which looks to be handled separately;

ENV RUNC_COMMIT=v1.0.0-rc10

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>
@thaJeztah thaJeztah force-pushed the update_docker_containerd branch from 2c07ef0 to 6c27d84 Compare April 22, 2021 07:03
@thaJeztah thaJeztah changed the title Update docker 20.10.6, containerd 1.4.4 Update docker 20.10.6 Apr 22, 2021
@thaJeztah
Copy link
Contributor Author

Reverted the containerd update 👍

@deitch
Copy link
Collaborator

deitch commented Apr 22, 2021

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?

@thaJeztah
Copy link
Contributor Author

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

@rgl
Copy link
Contributor

rgl commented May 30, 2021

Ping?

Maybe also include latest containerd 1.5.2?

PS: I'm looking forward for using cgroup v2!

@djs55
Copy link
Contributor

djs55 commented Jun 2, 2021

I think we can merge this now.

Since #3623 has been merged, we can make a separate PR for containerd 1.4.4 and one for runc as @deitch suggests above.

Edit: looks like containerd 1.4.4. was included in 95ce638

@djs55 djs55 merged commit 79b32dc into linuxkit:master Jun 2, 2021
@thaJeztah thaJeztah deleted the update_docker_containerd branch June 2, 2021 10:24
@thaJeztah
Copy link
Contributor Author

Edit: looks like containerd 1.4.4. was included in 95ce638

Probably should be updated to containerd 1.4.6 and runc v1.0.0-rc95

djs55 added a commit to djs55/linuxkit that referenced this pull request Jun 2, 2021
As suggested on linuxkit#3554 (comment)

Signed-off-by: David Scott <dave@recoil.org>
@djs55
Copy link
Contributor

djs55 commented Jun 2, 2021

@thaJeztah I've made separate PRs for containerd.1.4.6 and runc-v1.0.0-rc95 as you suggested!

rn pushed a commit to rn/linuxkit that referenced this pull request Jun 3, 2021
As suggested on linuxkit#3554 (comment)

Signed-off-by: David Scott <dave@recoil.org>
djs55 added a commit to djs55/linuxkit that referenced this pull request Oct 15, 2021
As suggested on linuxkit#3554 (comment)

Signed-off-by: David Scott <dave@recoil.org>
djs55 added a commit to djs55/linuxkit that referenced this pull request Oct 16, 2021
As suggested on linuxkit#3554 (comment)

Signed-off-by: David Scott <dave@recoil.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants