Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Add dependency on handlers role to docker role #1428

Merged
merged 1 commit into from
May 9, 2016

Conversation

langston-barrett
Copy link
Contributor

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

Useful if you run the Docker role by itself. Fixes this issue:

TASK: [docker | systemd drop-in for options] ********************************** 
ok: [lb0-worker-003] => (item=10-options.conf)
ok: [lb0-worker-001] => (item=10-options.conf)
ok: [lb0-kubeworker-002] => (item=10-options.conf)
ok: [lb0-worker-004] => (item=10-options.conf)
ok: [lb0-worker-002] => (item=10-options.conf)
ok: [lb0-kubeworker-001] => (item=10-options.conf)
ok: [lb0-edge-01] => (item=10-options.conf)
ok: [lb0-control-03] => (item=10-options.conf)
ok: [lb0-control-02] => (item=10-options.conf)
ok: [lb0-control-01] => (item=10-options.conf)
changed: [lb0-worker-003] => (item=12-network-options.conf)
changed: [lb0-worker-004] => (item=12-network-options.conf)
changed: [lb0-worker-001] => (item=12-network-options.conf)
changed: [lb0-worker-002] => (item=12-network-options.conf)
changed: [lb0-kubeworker-001] => (item=12-network-options.conf)
changed: [lb0-kubeworker-002] => (item=12-network-options.conf)
changed: [lb0-edge-01] => (item=12-network-options.conf)
changed: [lb0-control-01] => (item=12-network-options.conf)
changed: [lb0-control-03] => (item=12-network-options.conf)
changed: [lb0-control-02] => (item=12-network-options.conf)
ERROR: change handler (reload systemd) is not defined

@ryane
Copy link
Contributor

ryane commented May 9, 2016

lgtm, tested on aws

@ryane ryane merged commit 9964707 into master May 9, 2016
@ryane ryane deleted the fix/docker-role-handlers branch May 9, 2016 13:44
@ryane ryane added this to the 1.1 milestone May 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants