lp:~justin-fathomdb/nova/check-subprocess-exit-code
Created by
justinsb
and last modified
- Get this branch:
- bzr branch lp:~justin-fathomdb/nova/check-subprocess-exit-code
Branch merges
Propose for merging
No branches
dependent on this one.
- termie (community): Approve
- justinsb (community): Approve
- Vish Ishaya (community): Approve
-
Diff: 543 lines (+140/-135)12 files modifiedbin/nova-import-canonical-imagestore (+3/-3)
bin/nova-manage (+2/-1)
nova/cloudpipe/bootscript.sh (+2/-2)
nova/objectstore/image.py (+12/-3)
nova/process.py (+85/-100)
nova/tests/network_unittest.py (+0/-1)
nova/tests/process_unittest.py (+1/-1)
nova/utils.py (+14/-8)
nova/virt/images.py (+1/-1)
nova/virt/libvirt_conn.py (+3/-3)
nova/volume/service.py (+13/-7)
tools/install_venv.py (+4/-5)
Related source package recipes
Related bugs
| Bug #608505: Error in unit test NetworkTestCase test_subnet_edge | Medium | Fix Released |
|
Related blueprints
Branch information
Recent revisions
- 166. By justinsb
-
Fix unit test bug this uncovered: don't release_ip that we haven't got from issue_ip
- 165. By justinsb
-
Fix to better reflect (my believed intent) as to the meaning of error_ok (ignore stderr vs accept failure)
- 163. By Justin Santa Barbara <justinsb@justinsb-desktop>
-
Used new (clearer) flag names when calling processes
- 161. By Justin Santa Barbara <justinsb@justinsb-desktop>
-
Greater compliance with pep8/pylint style checks
- 159. By justinsb
-
Added exit code checking to process.py (twisted process utils). A bit of class refactoring to make it work & cleaner.
Also added some more instructive messages to install_venv.py, because otherwise people that don't know what they're doing will install the wrong pip... i.e. I did :-)
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:~hudson-openstack/nova/trunk
