Nir Soffer has posted comments on this change. Change subject: Live Merge: Suspend disk stats collection during pivot ......................................................................
Patch Set 8: (1 comment) http://gerrit.ovirt.org/#/c/31367/8/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 5642: if mode in (LiveMergeCleanupThread.MODE_CLEANUP, Line 5643: LiveMergeCleanupThread.MODE_PIVOT): Line 5644: self.log.info("Live merge job completed (job %s)", jobID) Line 5645: self._syncVolumeChain(drive) Line 5646: self.startDisksStatsCollection() > If pivot succeeds but cleanup fails you don't want to resume stats collecti Ok, makes sense for the disk being merged. But stats collection is done for all disks, so we don't want to stop it for other disks. I think we should not stop stats collections at all, but flag the disks which we are pivoting/cleaning up. Did you check direction? Line 5647: else: Line 5648: raise RuntimeError("Invalid mode: '%s'" % mode) Line 5649: Line 5650: def _initLegacyConf(self): -- To view, visit http://gerrit.ovirt.org/31367 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08689aec4d61871a568a6f92661b560fbf4d0b57 Gerrit-PatchSet: 8 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: Greg Padgett <[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
