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

Upgrade Docker to 18.09 (and containerd) #4180

Closed
afbjorklund opened this issue May 1, 2019 · 4 comments · Fixed by #4204
Closed

Upgrade Docker to 18.09 (and containerd) #4180

afbjorklund opened this issue May 1, 2019 · 4 comments · Fixed by #4204
Assignees
Labels
co/runtime/containerd co/runtime/docker Issues specific to a docker runtime kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Milestone

Comments

@afbjorklund
Copy link
Collaborator

While Kubernetes still recommends* using 18.06, some people are using the minikube Docker daemon for other purposes (such as building) and could benefit from upgrading to 18.09.

https://kubernetes.io/docs/setup/cri/#docker

Version 18.06.2 is recommended, but 1.11, 1.12, 1.13, 17.03 and 18.09 are known to work as well. Keep track of the latest verified Docker version in the Kubernetes release notes.

https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.14.md#kubernetes-v114-release-notes

The list of validated docker versions has changed. 1.11.1 and 1.12.1 have been removed. The current list is 1.13.1, 17.03, 17.06, 17.09, 18.06, 18.09.

As a part of this upgrade, we can stop using docker-runc and docker-containerd/docker-containerd-ctr and instead use the regular runc and containerd/ctr.

18.06.3-ce

docker/
docker/docker
docker/docker-containerd
docker/docker-containerd-ctr
docker/docker-containerd-shim
docker/dockerd
docker/docker-init
docker/docker-proxy
docker/docker-run

18.09.5

docker/
docker/containerd
docker/containerd-shim
docker/ctr
docker/docker
docker/dockerd
docker/docker-init
docker/docker-proxy
docker/runc

@afbjorklund afbjorklund added co/runtime/docker Issues specific to a docker runtime kind/feature Categorizes issue or PR as related to a new feature. labels May 1, 2019
@tstromberg
Copy link
Contributor

Related issue: #3519

@tstromberg tstromberg added the priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. label May 2, 2019
@afbjorklund
Copy link
Collaborator Author

Related issue: #3519

Can't we just delete the DOCKER_API_VERSION environment variable, I don't see why it is useful ?

The API will be upgraded automatically with the daemon, and downgraded gracefully with the client...

@afbjorklund
Copy link
Collaborator Author

afbjorklund commented May 4, 2019

Most likely, this also means that we need upgrade runc and containerd which we currently build.
(one day we should probably remove that misleading -bin from the package names, but whatever)

These are the versions that docker bundles (with 18.09.5):

runc version 1.0.0-rc6+dev
commit: 2b18fe1d885ee5083ef9f0838fee39b62d653e30
spec: 1.0.1-dev

containerd github.com/containerd/containerd v1.2.5 bb71b10fd8f58240ca47fbb579b9d1028eea7c84

And these are the versions minikube has:

# HEAD as of 2019-02-13
RUNC_MASTER_VERSION = 0a012df867a2d525f62a146d8ebdf2e6ab8a5ad5

CONTAINERD_BIN_VERSION = v1.2.0
CONTAINERD_BIN_COMMIT = c4446665cb9c30056f4998ed953e6d4ff22c7c39

That -master is probably redundant at best, and misleading at worst (since it has a fixed revision)
Normally one uses name such as -dev or -head or -master for things that build straight off HEAD

@afbjorklund afbjorklund changed the title Upgrade Docker to 18.09 Upgrade Docker to 18.09 (and containerd) May 4, 2019
@tstromberg tstromberg added this to the v1.1.0 milestone May 6, 2019
@afbjorklund afbjorklund self-assigned this May 6, 2019
@afbjorklund
Copy link
Collaborator Author

Should probably change to 18.09.6 ? (2019-05-06)

https://docs.docker.com/engine/release-notes/#18096

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
co/runtime/containerd co/runtime/docker Issues specific to a docker runtime kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants