Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
juju: only use configstore for bootstrap config #4490
Conversation
axw
added some commits
Feb 19, 2016
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Feb 22, 2016
jujubot
merged commit 75272f1
into
juju:master
Feb 22, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
axw commentedFeb 22, 2016
In juju/api.go, we now only use configstore for
the bootstrap config address resolution fallback.
juju.NewAPIState has been removed, and inlined
into the few tests that required it.
cmd/juju/register.go no longer calls
juju.UpdateControllerAddresses directly, but
first records the addresses used to contact the
controller in the first place, and then relies
on the proceeding API connection to update the
store.
Finally, we add a "switch-user" command, which
was the reason behind this branch in the first
place.
(Review request: http://reviews.vapour.ws/r/3923/)