Merge lp://qastaging/~e0ne/nova/associate-address-fix into lp://qastaging/~hudson-openstack/nova/trunk
Proposed by
Ivan Kolodyazhny
Status: | Work in progress |
---|---|
Proposed branch: | lp://qastaging/~e0ne/nova/associate-address-fix |
Merge into: | lp://qastaging/~hudson-openstack/nova/trunk |
Diff against target: |
41 lines (+7/-2) 3 files modified
.mailmap (+1/-0) Authors (+1/-0) bin/nova-manage (+5/-2) |
To merge this branch: | bzr merge lp://qastaging/~e0ne/nova/associate-address-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Josh Kearney (community) | Approve | ||
Matt Dietz (community) | Approve | ||
Mandell (community) | community | Approve | |
Vish Ishaya (community) | Needs Fixing | ||
Devin Carlen (community) | Approve | ||
Masanori Itoh | Pending | ||
Review via email:
|
Description of the change
Please, review my changes. I fixed Bug #751229
To post a comment you must log in.
Unmerged revisions
- 1212. By ikolodyazhny <ikolodyazhny@ikolodyazhny-laptop>
-
Merge conflicts fixed
- 1211. By ikolodyazhny <ikolodyazhny@ikolodyazhny-laptop>
-
Merge with lp:nova
- 1210. By Ivan Kolodyazhny
-
mailmap and Authors files updated
- 1209. By Ivan Kolodyazhny
-
Change code to use netaddr instead of IPy
- 1208. By Ivan Kolodyazhny
-
Merged with trunk
- 1207. By e0ne <e0ne@e0ne-vm>
-
Move context.
get_admin_ context( ) call outside of the loop - 1206. By e0ne <e0ne@e0ne-vm>
-
Typo fixed
- 1205. By e0ne <e0ne@e0ne-vm>
-
Floating address range fixed
While you're fixing the code, you should probably move the call to context. get_admin_ context( ) outside of the loop. No need to run it several times.