Merge lp:~dan-prince/nova/fix_allocate_fixed_ips into lp:~hudson-openstack/nova/trunk
Proposed by
Dan Prince
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Dan Prince | ||||
Approved revision: | 1269 | ||||
Merged at revision: | 1268 | ||||
Proposed branch: | lp:~dan-prince/nova/fix_allocate_fixed_ips | ||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||
Diff against target: | 0 lines | ||||
To merge this branch: | bzr merge lp:~dan-prince/nova/fix_allocate_fixed_ips | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vish Ishaya (community) | Approve | ||
Mohammed Naser (community) | Approve | ||
Review via email: mp+67777@code.launchpad.net |
Commit message
Don't pop 'vpn' on kwargs inside a loop in RPCAllocateFixe
Update allocate_fixed_ip methods in network manager to accept correct kwargs. (fixes regression in bzr 1266)
To post a comment you must log in.
Code looks good and works again for me! :)