Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
apiserver/client: InstanceConfig: use cached addrs #4816
Conversation
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Mar 21, 2016
jujubot
merged commit ecdb7b0
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
axw commentedMar 21, 2016
The ProvisioningScript client method calls InstanceConfig,
which was constructing an Environ to get the Juju API
addresses. We already store those addresses in state, so
just fetch them from state.
Fixes https://bugs.launchpad.net/juju-core/+bug/1559299
(Review request: http://reviews.vapour.ws/r/4258/)