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@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Royce Lv lvroyce@linux.vnet.ibm.com