Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

Update docker.service to work with 1.12.0 #90

Merged
merged 2 commits into from
Aug 3, 2016

Conversation

manics
Copy link
Member

@manics manics commented Aug 2, 2016

The upstream docker service file has changed with 1.12 (no longer using a separate service for the socket)

Also makes role: system-monitor-agent part of the standard install for all idr servers idrsystems-playbooks/idr-initial.yml instead of for individual groups.

@atarkowska
Copy link
Member

atarkowska commented Aug 2, 2016

tested today with new vm

$ source ~/OMERO/VENV/ansible/bin/activate
(ansible) $ ansible-playbook -i infrastructure/ansible/inventory/openstack.py -l ola-mapr -vv -u centos infrastructure/openstack/examples/omedev-docker.yml
(ansible) $ ansible-playbook -i infrastructure/ansible/inventory/openstack.py -l ola-mapr -vv -u centos infrastructure/openstack/examples/devspace.yml

https://github.com/aleksandra-tarkowska/infrastructure/blob/devspace/openstack/examples/omedev-docker.yml
https://github.com/aleksandra-tarkowska/infrastructure/blob/devspace/openstack/examples/devspace.yml

withouth this PR I got:

TASK [docker : docker | enable] ************************************************
task path: /Users/ola/OMERO/DOCKER/infrastructure/ansible/roles/docker/tasks/main.yml:55
fatal: [61fa3b27-5bb4-456c-b44c-2e826edcee6e]: FAILED! => {"changed": false, "failed": true, "msg": "Failed to start docker.service: Unit docker.socket failed to load: No such file or directory.\n"}

brand new 133 is back to life now

@kennethgillen
Copy link
Member

👍 thanks for the testing note @aleksandra-tarkowska

@kennethgillen kennethgillen merged commit 187d58b into ome:master Aug 3, 2016
@manics manics deleted the docker-1.12 branch August 3, 2016 10:06
manics pushed a commit to manics-archive/ome-infrastructure that referenced this pull request Nov 24, 2016
Update docker.service to work with 1.12.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants