/juju

instancepoller: don't back off until provisioned #6037

Merged
merged 1 commit into from Aug 19, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@axw @wallyworld @jujubot
@axw
Member

axw commented Aug 19, 2016

While the machine instance is not provisioned,
don't back off. This prevents the poller from
taking a long time to set the public addresses
when provisioning takes a long time.

Fixes https://bugs.launchpad.net/juju-core/+bug/1614749
@axw
instancepoller: don't back off until provisioned 
While the machine instance is not provisioned,
don't back off. This prevents the poller from
taking a long time to set the public addresses
when provisioning takes a long time.

Fixes https://bugs.launchpad.net/juju-core/+bug/1614749
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
ed92c6f
@axw
Member

axw commented Aug 19, 2016

QA

  1. bootstrap azure
  2. add-machine
  3. watch status, observe that machine's public address shows up in status ~immediately after it transitions to "started"
@wallyworld
Owner

wallyworld commented Aug 19, 2016

LGTM
@axw
Member

axw commented Aug 19, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Aug 19, 2016

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

@jujubot jujubot merged commit 1837799 into juju:master Aug 19, 2016

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