Change in vdsm[master]: vm: refresh all volumes after migration

abaron at redhat.com abaron at redhat.com
Thu Oct 18 12:14:52 UTC 2012


Ayal Baron has posted comments on this change.

Change subject: vm: refresh all volumes after migration
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/libvirtvm.py
Line 1637:     def refreshAllDrives(self):
Line 1638:         volInfoList = []
Line 1639: 
Line 1640:         for device in self._devices[vm.DISK_DEVICES][:]:
Line 1641:             if not device.blockDev or not hasattr(device, 'domainID'):
since this test is repeated in several places, please consider refactoring it into a method (other places are possibly currently missing the domainID check even though they should check for it)
Line 1642:                 continue
Line 1643: 
Line 1644:             volInfoList.append({
Line 1645:                 'domainID': device.domainID,


--
To view, visit http://gerrit.ovirt.org/8518
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibe0bd784e68f162b7902bb6ae50ff183d9f8fa8a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel P. Berrange <berrange at redhat.com>


More information about the vdsm-patches mailing list