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 <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Royce Lv <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
