Dynamically discover init system in the cloudinit script. #2358

Merged
merged 6 commits into from May 19, 2015

Conversation

Projects
None yet
2 participants
Contributor

ericsnowcurrently commented May 19, 2015

(fixes https://bugs.launchpad.net/juju-core/+bug/1450092)

Keying off the juju version doesn't work well for local provider. This patch resolves the issue by adding discovery to the cloudinit script.

(Review request: http://reviews.vapour.ws/r/1595/)

Contributor

ericsnowcurrently commented May 19, 2015

$$merge$$

Contributor

jujubot commented May 19, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request May 19, 2015

Merge pull request #2358 from ericsnowcurrently/fix-1450092-cloudinit…
…-init-system-detection

Dynamically discover init system in the cloudinit script.

(fixes https://bugs.launchpad.net/juju-core/+bug/1450092)

Keying off the juju version doesn't work well for local provider. This patch resolves the issue by adding discovery to the cloudinit script.

(Review request: http://reviews.vapour.ws/r/1595/)

@jujubot jujubot merged commit ebfc363 into juju:master May 19, 2015

@ericsnowcurrently ericsnowcurrently deleted the ericsnowcurrently:fix-1450092-cloudinit-init-system-detection branch May 19, 2015

ericsnowcurrently added a commit to ericsnowcurrently/juju that referenced this pull request May 21, 2015

Revert "Merge pull request #2358 from ericsnowcurrently/fix-1450092-c…
…loudinit-init-system-detection"

This reverts commit ebfc363, reversing
changes made to f1f008c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment