adt-cloud-worker

Merge lp://staging/~fginther/adt-cloud-worker/uci-nova-wait-for-active into lp://staging/~canonical-ci-engineering/adt-cloud-worker/uci-nova

  1. uci-nova-wait-for-active
  2. Merge into uci-nova
Proposed by Francis Ginther
Status: Merged
Approved by: Francis Ginther
Approved revision: 13
Merged at revision: 10
Proposed branch: lp://staging/~fginther/adt-cloud-worker/uci-nova-wait-for-active
Merge into: lp://staging/~canonical-ci-engineering/adt-cloud-worker/uci-nova
Prerequisite: lp://staging/~cprov/adt-cloud-worker/uci-nova-refix
Diff against target: 16 lines (+5/-1)
1 file modified
uci-nova (+5/-1)
To merge this branch: bzr merge lp://staging/~fginther/adt-cloud-worker/uci-nova-wait-for-active
Reviewer Review Type Date Requested Status
Celso Providelo (community) Approve
Review via email: mp+256877@code.staging.launchpad.net

Commit message

Wait for nova instance to become ACTIVE before searching for ipaddr.

Description of the change

Wait for nova instance to become ACTIVE before searching for ipaddr.

This solves the problem of other 'network' text in the "nova show" output tripping up the awk command searching for ipaddr.

https://trello.com/c/WA8HuriB/68-proposed-migration-uci-nova-can-get-tripped-up-when-looking-for-the-ip-address

To post a comment you must log in.
lp://staging/~fginther/adt-cloud-worker/uci-nova-wait-for-active updated
13. By Francis Ginther

Add a comment on why the ACTIVE check was added.

Revision history for this message
Celso Providelo (cprov) wrote :

Thanks for fixing this subtle problem, Francis.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches