Differential D987

Switch Docker Engine to docker-ce official repository on CentOS
ClosedPublic
Actions

Authored by dereckson on May 24 2017, 10:32.

Details

Reviewers

dereckson

Maniphest Tasks

T1205: Upgrade Docker Engine to 1.13+ on Dwellers

Commits

rOPSb2d950dd11a7: Switch Docker Engine to docker-ce official repository on CentOS

Summary

Previously, Docker was installed from two repositories:

EPEL
virt7, a specialized CBS on CentOS from Project Atomic.

We now switch to use the docker-ce CentOS repository from Docker.

Reference:
https://docs.docker.com/engine/installation/linux/centos/#install-using-the-repository

Test Plan

State applied on Dwellers.

Dwellers already have this update as it was applied manually yesterday
to unstuck Mastodon containers (the docker-compose version required
Docker 1.13+, CBS still provides 1.12).

     ID: remove_legacy_docker_packages
Comment: All specified packages are already absent

     ID: install_docker_engine
Comment: All specified packages are already installed

So it's correctly, as expected no-op.

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

docker (branched from master)

Build Status

Buildable 1535
Build 1783: arc lint + arc unit

Event Timeline

dereckson created this revision.May 24 2017, 10:32

dereckson planned changes to this revision.May 24 2017, 10:34

dereckson added inline comments.

roles/paas-docker/docker/init.sls
29	We also need to saltize the docker-ce repository. URL: https://download.docker.com/linux/centos/docker-ce.repo Current configuration file in /etc/yum.repos.d/docker-ce.repo.

Download docker-ce.repo

We can't use pkgrepo conveniently, as it's a state to manually populate a repo.

      ID: install_docker_engine
Function: file.managed
    Name: /etc/yum.repos.d/docker-ce.repo
  Result: True
 Comment: File /etc/yum.repos.d/docker-ce.repo is in the correct state
 Started: 12:42:02.023580
Duration: 533.147 ms
 Changes:

dereckson accepted this revision.May 24 2017, 10:43

dereckson marked an inline comment as done.

This revision is now accepted and ready to land.May 24 2017, 10:43