initial fudging in of swap disk

2 files changed, 25 insertions, 14 deletions

diff --git a/nova/virt/xenapi/vm_utils.py b/nova/virt/xenapi/vm_utils.py
index 9f6cd608c..c24fc7ba6 100644
--- a/nova/virt/xenapi/vm_utils.py
+++ b/nova/virt/xenapi/vm_utils.py
@@ -408,18 +408,24 @@ class VMHelper(HelperBase):
 
         kwargs = {'params': pickle.dumps(params)}
         task = session.async_call_plugin('glance', 'download_vhd', kwargs)
-        vdi_uuid = session.wait_for_task(task, instance_id)
+        vdi_uuids = session.wait_for_task(task, instance_id)
+        primary_vdi_uuid = vdi_uuids.get('primary_vdi_uuid')
+        swap_vdi_uuid = vdi_uuids.get('swap_vdi_uuid')
 
         cls.scan_sr(session, instance_id, sr_ref)
 
         # Set the name-label to ease debugging
-        vdi_ref = session.get_xenapi().VDI.get_by_uuid(vdi_uuid)
-        name_label = get_name_label_for_image(image)
-        session.get_xenapi().VDI.set_name_label(vdi_ref, name_label)
+        primary_vdi_ref = session.get_xenapi().VDI.get_by_uuid(primary_vdi_uuid)
+        primary_name_label = get_name_label_for_image(image)
+        session.get_xenapi().VDI.set_name_label(primary_vdi_ref, primary_name_label)
 
-        LOG.debug(_("xapi 'download_vhd' returned VDI UUID %(vdi_uuid)s")
+        LOG.debug(_("xapi 'download_vhd' returned VDI UUID %(primary_vdi_uuid)s")
                   % locals())
-        return vdi_uuid
+
+	LOG.debug("=" * 100)
+        LOG.debug(rimary_vdi_uuid)
+        LOG.debug(swap_vdi_uuid)
+        return (primary_vdi_uuid, swap_vdi_uuid)
 
     @classmethod
     def _fetch_image_glance_disk(cls, session, instance_id, image, access,
diff --git a/nova/virt/xenapi/vmops.py b/nova/virt/xenapi/vmops.py
index 0074444f8..4a01cac29 100644
--- a/nova/virt/xenapi/vmops.py
+++ b/nova/virt/xenapi/vmops.py
@@ -109,20 +109,20 @@ class VMOps(object):
         user = AuthManager().get_user(instance.user_id)
         project = AuthManager().get_project(instance.project_id)
         disk_image_type = VMHelper.determine_disk_image_type(instance)
-        vdi_uuid = VMHelper.fetch_image(self._session, instance.id,
-                instance.image_id, user, project, disk_image_type)
-        return vdi_uuid
+        (primary_vdi_uuid, swap_vdi_uuid) = VMHelper.fetch_image(self._session,
+                instance.id, instance.image_id, user, project, disk_image_type)
+        return (primary_vdi_uuid, swap_vdi_uuid)
 
     def spawn(self, instance, network_info=None):
-        vdi_uuid = self._create_disk(instance)
-        vm_ref = self._create_vm(instance, vdi_uuid, network_info)
+        vdi_uuid, swap_uuid = self._create_disk(instance)
+        vm_ref = self._create_vm(instance, vdi_uuid, swap_uuid, network_info)
         self._spawn(instance, vm_ref)
 
     def spawn_rescue(self, instance):
         """Spawn a rescue instance."""
         self.spawn(instance)
 
-    def _create_vm(self, instance, vdi_uuid, network_info=None):
+    def _create_vm(self, instance, vdi_uuid, swap_vdi_uuid=None, network_info=None):
         """Create VM instance."""
         instance_name = instance.name
         vm_ref = VMHelper.lookup(self._session, instance_name)
@@ -143,18 +143,20 @@ class VMOps(object):
 
         # Are we building from a pre-existing disk?
         vdi_ref = self._session.call_xenapi('VDI.get_by_uuid', vdi_uuid)
+        if swap_vdi_uuid:
+            swap_vdi_ref = self._session.call_xenapi('VDI.get_by_uuid', swap_vdi_uuid)
 
         disk_image_type = VMHelper.determine_disk_image_type(instance)
 
         kernel = None
         if instance.kernel_id:
             kernel = VMHelper.fetch_image(self._session, instance.id,
-                instance.kernel_id, user, project, ImageType.KERNEL_RAMDISK)
+                instance.kernel_id, user, project, ImageType.KERNEL_RAMDISK)[0]
 
         ramdisk = None
         if instance.ramdisk_id:
             ramdisk = VMHelper.fetch_image(self._session, instance.id,
-                instance.ramdisk_id, user, project, ImageType.KERNEL_RAMDISK)
+                instance.ramdisk_id, user, project, ImageType.KERNEL_RAMDISK)[0]
 
         use_pv_kernel = VMHelper.determine_is_pv(self._session, instance.id,
             vdi_ref, disk_image_type, instance.os_type)
@@ -163,6 +165,9 @@ class VMOps(object):
 
         VMHelper.create_vbd(session=self._session, vm_ref=vm_ref,
                 vdi_ref=vdi_ref, userdevice=0, bootable=True)
+	if swap_vdi_uuid:
+            VMHelper.create_vbd(session=self._session, vm_ref=vm_ref,
+                    vdi_ref=swap_vdi_ref, userdevice=0, bootable=False)
 
         # TODO(tr3buchet) - check to make sure we have network info, otherwise
         # create it now. This goes away once nova-multi-nic hits.