[cern] upgrade docker-ce version in k8s

Add label to specify the docker-ce version to use in a kubernetes cluster. This will replace the available dockerd in the image, which is often quite old.

Fixes OS-7273.

the patch looks good.

The only issue that we need to keep in mind is that docker will be already running and probably will have some containers, coredns or dashboard/heapster.

Since the script is small we may be able to keep it as a software config in user data before enabling services [0].

I don't know how we can test outside devstack, maybe teststack.

[0] https://gitlab.cern.ch/cloud-infrastructure/magnum/blob/cern-queens/magnum/drivers/k8s_fedora_atomic_v1/templates/kubemaster.yaml#L679

I wanted to leave a note and forgot, regarding that exactly. There's no way we can have these bits ordered using softwaredeployments, right?

[k8s] Set order in kubemaster software deployments https://review.openstack.org/#/c/591592/

Would it work even with some softwareconfigs left? Or do i need to change it to a softwareconfig for now?

if you add it a as software config bedore coredns (in our local branch) I think it will work

I would try putting the software config here: https://gitlab.cern.ch/cloud-infrastructure/magnum/blob/cern-queens/magnum/drivers/k8s_fedora_atomic_v1/templates/kubemaster.yaml#L679

added 1 commit

• 2d37a519 - [cern] upgrade docker-ce version in k8s

Compare with previous version

lgtm

approved this merge request

added 2 commits

• e05fba3e - 1 commit from branch cern-queens
• 7747c8a6 - [cern] upgrade docker-ce version in k8s

Compare with previous version

lgtm

closed