adt-cloud-worker

Merge lp://staging/~cprov/adt-cloud-worker/uci-nova-neutron-fallback into lp://staging/~canonical-ci-engineering/adt-cloud-worker/uci-nova

  1. uci-nova-neutron-fallback
  2. Merge into uci-nova
Proposed by Celso Providelo
Status: Merged
Merged at revision: 2
Proposed branch: lp://staging/~cprov/adt-cloud-worker/uci-nova-neutron-fallback
Merge into: lp://staging/~canonical-ci-engineering/adt-cloud-worker/uci-nova
Diff against target: 106 lines (+46/-20)
1 file modified
uci-nova (+46/-20)
To merge this branch: bzr merge lp://staging/~cprov/adt-cloud-worker/uci-nova-neutron-fallback
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+253147@code.staging.launchpad.net

Commit message

Coping with missing neutron security-groups capabilities (cannonistack and stackystack) and falling back to nova security-groups without egress traffic control.

Description of the change

Coping with missing neutron security-groups capabilities (cannonistack and stackystack) and falling back to nova security-groups without egress traffic control.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

Tested against both canonistack and bootstack. Bootstack blocked egress traffic as expected, canonistack not so much. Both blocked ingress traffic as expected.

Just one question inline.

review: Needs Information
Revision history for this message
Francis Ginther (fginther) wrote :

Why was --poll removed?

"because it breaks on slow clouds (by reaching nova controller too often) and also because it wasn't really necessary, the scripts polls for ip-address and also ssh access moments later"

Thanks

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