lp:~markmc/nova/iscsi-tgtadm-choice
- Get this branch:
- bzr branch lp:~markmc/nova/iscsi-tgtadm-choice
Branch merges
- Christopher MacGown (community): Approve
-
Diff: 553 lines (+331/-76)4 files modifiednova/tests/test_iscsi.py (+116/-0)
nova/tests/test_volume.py (+6/-11)
nova/volume/driver.py (+53/-65)
nova/volume/iscsi.py (+156/-0)
Related bugs
Bug #819997: Nova volume should support tgt rather than iscsitarget. | Wishlist | Fix Released |
Related blueprints
Branch information
Recent revisions
- 1422. By Mark McLoughlin
-
Refactor ietadm/tgtadm calls out into helper classes
Add a new TargetAdmin abstract base class and implement it using ietadm
and tgtadm. This cleans up the code greatly and gets us some code reuse. - 1421. By Mark McLoughlin
-
Remove VolumeDriver.
sync_exec method We always use the same functions for sync_exec and execute.
The execute method is always synchronous, so the distinction doesn't
appear to make sense.Finally, it looks like it would make sense for execute to ever be
async, so the distinction isn't even serving a useful documentation
purpose. - 1414. By Ed Leafe
-
Removed verbose debugging output when capabilities are reported. This was clogging up the logs with kbytes of useless data, preventing actual helpful information from being retrieved easily.
- 1413. By Matthew Hooker
-
These fixes are the result of trolling the pylint violations here
https:/
/jenkins. openstack. org/job/ nova-pylint- errors/ violations/
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