provider/manual: drop use-sshstorage #4817

Merged
merged 1 commit into from Mar 21, 2016

Conversation

Projects
None yet
3 participants
Member

axw commented Mar 21, 2016

We were using use-sshstorage in one place:
to decide whether ControllerInstances should
perform a host validity check or not. Instead
of doing that, we can just check if we're
running inside "jujud". With that in place,
drop use-sshstorage from config.

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

provider/manual: drop use-sshstorage
We were using use-sshstorage in one place:
to decide whether ControllerInstances should
perform a host validity check or not. Instead
of doing that, we can just check if we're
running inside "jujud". With that in place,
drop use-sshstorage from config.
Owner

wallyworld commented Mar 21, 2016

$$merge$$

Contributor

jujubot commented Mar 21, 2016

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

jujubot added a commit that referenced this pull request Mar 21, 2016

Merge pull request #4817 from axw/manual-drop-usesshstorage
provider/manual: drop use-sshstorage

We were using use-sshstorage in one place:
to decide whether ControllerInstances should
perform a host validity check or not. Instead
of doing that, we can just check if we're
running inside "jujud". With that in place,
drop use-sshstorage from config.

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

@jujubot jujubot merged commit ce509cd into juju:admin-controller-model Mar 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment