Merge pull request #283 from xcp-ng/fix-vm-vdi_uuids

Fix `VM.vdi_uuids`

Author: ydirson

lib/basevm.py

@@ -40,9 +40,10 @@ def name(self):
 
     def vdi_uuids(self, sr_uuid=None):
         output = self._disk_list()
-        all_uuids = output.split(',')  # Split based on commas
-        # Select only every alternate UUID (even index positions)
-        vdis = all_uuids[0::2]  # Start at 0, step by 2
+        if output == '':
+            return []
+
+        vdis = output.split(',')
 
         if sr_uuid is None:
             return vdis

lib/snapshot.py

@@ -4,7 +4,8 @@
 
 class Snapshot(BaseVM):
     def _disk_list(self):
-        return self.host.xe('snapshot-disk-list', {'uuid': self.uuid}, minimal=True)
+        return self.host.xe('snapshot-disk-list', {'uuid': self.uuid, 'vbd-params': ''},
+                            minimal=True)
 
     def destroy(self, verify=False):
         logging.info("Delete snapshot " + self.uuid)

lib/vm.py

@@ -159,7 +159,7 @@ def unpause(self):
         self.host.xe('vm-unpause', {'uuid': self.uuid})
 
     def _disk_list(self):
-        return self.host.xe('vm-disk-list', {'uuid': self.uuid}, minimal=True)
+        return self.host.xe('vm-disk-list', {'uuid': self.uuid, 'vbd-params': ''}, minimal=True)
 
     def _destroy(self):
         self.host.xe('vm-destroy', {'uuid': self.uuid})