Francesco Romani has posted comments on this change. Change subject: Live Merge: Mark active layer ILLEGAL before pivoting ......................................................................
Patch Set 2: Code-Review+1 (1 comment) http://gerrit.ovirt.org/#/c/31787/2/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 5714: """ Line 5715: Returns True if the last cleanup phase succeeded and the job can be Line 5716: removed. Line 5717: """ Line 5718: return self.success and self.mode == self.MODE_CLEANUP I assume this hunk will be used by a later patch as you say in the commit message. Line 5719: Line 5720: Line 5721: def _devicesWithAlias(domXML): Line 5722: return vmxml.filter_devices_with_alias(vmxml.all_devices(domXML)) -- To view, visit http://gerrit.ovirt.org/31787 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I97ce8fbdfad7c81181a206f160ffdd647c552d4d Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
