Remove python-losetup dependency
[snf-image-creator] / image_creator / disk.py
index 6f219bf..809a1a3 100644 (file)
@@ -1,6 +1,8 @@
 #!/usr/bin/env python
 
-import losetup
+from image_creator.util import get_command
+from clint.textui import progress
+
 import stat
 import os
 import tempfile
@@ -9,18 +11,27 @@ import re
 import sys
 import guestfs
 
-from pbs import dmsetup
-from pbs import blockdev
-from pbs import dd
-
 
 class DiskError(Exception):
     pass
 
+dd = get_command('dd')
+dmsetup = get_command('dmsetup')
+losetup = get_command('losetup')
+blockdev = get_command('blockdev')
+
 
 class Disk(object):
+    """This class represents a hard disk hosting an Operating System
+
+    A Disk instance never alters the source media it is created from.
+    Any change is done on a snapshot created by the device-mapper of
+    the Linux kernel.
+    """
 
     def __init__(self, source):
+        """Create a new Disk instance out of a source media. The source
+        media can be an image file, a block device or a directory."""
         self._cleanup_jobs = []
         self._devices = []
         self.source = source
@@ -29,15 +40,18 @@ class Disk(object):
         self._cleanup_jobs.append((job, args))
 
     def _losetup(self, fname):
-        loop = losetup.find_unused_loop_device()
-        loop.mount(fname)
-        self._add_cleanup(loop.unmount)
-        return loop.device
+        loop = losetup('-f', '--show', fname)
+        loop = loop.strip() # remove the new-line char
+        self._add_cleanup(losetup, '-d', loop)
+        return loop
 
     def _dir_to_disk(self):
         raise NotImplementedError
 
     def cleanup(self):
+        """Cleanup internal data. This needs to be called before the
+        program ends.
+        """
         while len(self._devices):
             device = self._devices.pop()
             device.destroy()
@@ -47,6 +61,11 @@ class Disk(object):
             job(*args)
 
     def get_device(self):
+        """Returns a newly created DiskDevice instance.
+
+        This instance is a snapshot of the original source media of
+        the Disk instance.
+        """
         sourcedev = self.source
         mode = os.stat(self.source).st_mode
         if stat.S_ISDIR(mode):
@@ -79,22 +98,43 @@ class Disk(object):
         return new_device
 
     def destroy_device(self, device):
+        """Destroys a DiskDevice instance previously created by
+        get_device method.
+        """
         self._devices.remove(device)
         device.destroy()
 
 
+def progress_generator(total):
+    position = 0;
+    for i in progress.bar(range(total)):
+        if i < position:
+            continue
+        position = yield
+    yield #suppress the StopIteration exception
+
+
 class DiskDevice(object):
+    """This class represents a block device hosting an Operating System
+    as created by the device-mapper.
+    """
 
     def __init__(self, device, bootable=True):
+        """Create a new DiskDevice."""
         self.device = device
         self.bootable = bootable
+        self.progress_bar = None
 
         self.g = guestfs.GuestFS()
+        self.g.add_drive_opts(device, readonly=0)
 
-        self.g.set_trace(1)
+        #self.g.set_trace(1)
+        #self.g.set_verbose(1)
 
-        self.g.add_drive_opts(device, readonly=0)
+        eh = self.g.set_event_callback(self.progress_callback, guestfs.EVENT_PROGRESS)
         self.g.launch()
+        self.g.delete_event_callback(eh)
+        
         roots = self.g.inspect_os()
         if len(roots) == 0:
             raise DiskError("No operating system found")
@@ -106,13 +146,27 @@ class DiskDevice(object):
         self.distro = self.g.inspect_get_distro(self.root)
 
     def destroy(self):
+        """Destroy this DiskDevice instance."""
         self.g.umount_all()
         self.g.sync()
         # Close the guestfs handler
         self.g.close()
-        del self.g
+
+    def progress_callback(self, ev, eh, buf, array):
+        position = array[2]
+        total = array[3]
+        
+        if self.progress_bar is None:
+            self.progress_bar = progress_generator(total)
+            self.progress_bar.next()
+
+        self.progress_bar.send(position)
+
+        if position == total:
+            self.progress_bar = None
 
     def mount(self):
+        """Mount all disk partitions in a correct order."""
         mps = self.g.inspect_get_mountpoints(self.root)
 
         # Sort the keys to mount the fs in a correct order.
@@ -132,9 +186,16 @@ class DiskDevice(object):
                 print "%s (ignored)" % msg
 
     def umount(self):
+        """Umount all mounted filesystems."""
         self.g.umount_all()
 
     def shrink(self):
+        """Shrink the disk.
+
+        This is accomplished by shrinking the last filesystem in the
+        disk and then updating the partition table. The new disk size
+        (in bytes) is returned.
+        """
         dev = self.g.part_to_dev(self.root)
         parttype = self.g.part_get_parttype(dev)
         if parttype != 'msdos':
@@ -150,7 +211,7 @@ class DiskDevice(object):
         part_dev = "%s%d" % (dev, last_partition['part_num'])
         fs_type = self.g.vfs_type(part_dev)
         if not re.match("ext[234]", fs_type):
-            print "Warning, don't know how to resize %s partitions" % vfs_type
+            print "Warning: Don't know how to resize %s partitions." % vfs_type
             return
 
         self.g.e2fsck_f(part_dev)
@@ -164,8 +225,20 @@ class DiskDevice(object):
         start = last_partition['part_start'] / sector_size
         end = start + (block_size * block_cnt) / sector_size - 1
 
+        self.g.part_del(dev, last_partition['part_num'])
+        self.g.part_add(dev, 'p', start, end)
+
         return (end + 1) * sector_size
 
+    def size(self):
+        """Returns the "payload" size of the device.
+
+        The size returned by this method is the size of the space occupied by
+        the partitions (including the space before the first partition).
+        """
+        dev = self.g.part_to_dev(self.root)
+        last = self.g.part_list(dev)[-1]
 
+        return last['part_end']
 
 # vim: set sta sts=4 shiftwidth=4 sw=4 et ai :