Change in vdsm[master]: fix check None sdUUID for getVmsList and getVmsInfo

lvroyce at linux.vnet.ibm.com lvroyce at linux.vnet.ibm.com
Tue Apr 17 08:03:44 UTC 2012


Royce Lv has posted comments on this change.

Change subject: fix check None sdUUID for getVmsList and getVmsInfo
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File vdsm/storage/hsm.py
Line 1136:             self.validateBackupDom(sdUUID)
so if we store OVFs on all domains,
when we call getVmsList with no sdUUID specified,
 it means we should enumerate all storage domains of the storage pool
 to see whether it has OVFs on it, right?
Now we just enumerate master domain in this case.
At this moment, shall I change it or leave it what it is?
Thank you for suggestion: )

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I27fe906e6738320279bc44f9736acaa97131d7ee
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Royce Lv <lvroyce at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list