Merge lp://qastaging/~bgh/nova/qmanager-ipam-fixes into lp://qastaging/nova/essex

Proposed by Brad Hall
Status: Needs review
Proposed branch: lp://qastaging/~bgh/nova/qmanager-ipam-fixes
Merge into: lp://qastaging/nova/essex
Diff against target: 284 lines (+85/-72)
6 files modified
nova/network/quantum/client.py (+0/-2)
nova/network/quantum/manager.py (+42/-22)
nova/network/quantum/melange_ipam_lib.py (+25/-26)
nova/network/quantum/nova_ipam_lib.py (+3/-2)
nova/network/quantum/quantum_connection.py (+10/-14)
nova/tests/test_quantum.py (+5/-6)
To merge this branch: bzr merge lp://qastaging/~bgh/nova/qmanager-ipam-fixes
Reviewer Review Type Date Requested Status
Nova Core security contacts Pending
Review via email: mp+77438@code.qastaging.launchpad.net

Description of the change

A couple fixes to the currently experimental QuantumManager (network manager)

To post a comment you must log in.

Unmerged revisions

1647. By Brad Hall

Make sure the network info structure we're returning has a label

1646. By Brad Hall

Fix quantum/melange ipam interaction

We now query for the subnets by net_id/vif_id instead of searching through all
the blocks to find the right one. Both of the allocate and deallocate for
instance calls are now using the vif_id -> network_id mapping instead of
searching the quantum networks. get_port_by_attachment was also changed to
take a net_id so that we don't have to search through all of the quantum
networks to find the corresponding port.

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