Code Review

Bug#898257 abstract out disk image access methods 93/1993/7
authorPádraig Brady <pbrady@redhat.com>
Wed, 30 Nov 2011 17:00:17 +0000 (17:00 +0000)
committerPádraig Brady <P@draigBrady.com>
Sun, 11 Dec 2011 22:56:11 +0000 (22:56 +0000)
commitf97c6652cf0b8df64ac76b125dcbff413199f914
tree69eaf9f0e298408e5792e54499ca289a5ceb3d6a
parentce2d62f95cd9b62858c9b4ef37b418881ceaef07
Bug#898257 abstract out disk image access methods

Rather than providing two mutually exlusive image
access methods (loop and qemu-nbd), try each in turn.
This is to prepare for a follow up patch which will
add libguestfs as a method to try.

* nova/virt/mount.py: A new Mount class to abstract the
devce allocation, partition mapping and file sys mounting,
for each access type.
* nova/virt/disk/loop.py: A specialization of the base Mount class
to provide loop back mounting support.
* nova/virt/disk/nbd.py: A specialization of the base Mount class
to provide qemu-nbd mounting support.
* nova/virt/disk/base.py(img_handlers): A new list of access methods
to try, with the order being honored.
(_DiskImage): An internal helper class that uses the plugin classes
above, to provide the operations available on a disk image file.
When mounting, iterate over each access method until one succeeds.
If a hint is provided about a CoW format image, the list of
methods to try will be reduced accordingly.
Note expected errors are no longer raised as exceptions during mounting.
Instead, on failure to mount an image, errors are collated and raised.
Interveining errors are logged in debug mode for successful mounts.
* nova/virt/libvirt/connection.py: Adjust the function parameter
names to be more general, rather than referencing specific
implementations like 'nbd' and 'tune2fs'.
Simplify the destroy_container() by storing and passing
back a reference to the _DiskImage object, which has the
necessary state to unmount.
* nova/utils.py (trycmd): A helper function to both deal with,
commands that issue ignorable warnings to stderr,
and commands that EXIT_SUCCESS while issuing errors to stderr.

Change-Id: If3a4b1c8f4e2f2e7300a21071340dcc839cb36d7
Authors
nova/rootwrap/compute.py
nova/utils.py
nova/virt/disk/__init__.py [new file with mode: 0644]
nova/virt/disk/base.py [moved from nova/virt/disk.py with 57% similarity]
nova/virt/disk/loop.py [new file with mode: 0644]
nova/virt/disk/mount.py [new file with mode: 0644]
nova/virt/disk/nbd.py [new file with mode: 0644]
nova/virt/libvirt/connection.py