Dan Kenigsberg has posted comments on this change. Change subject: cancel migration: remove event sync ......................................................................
Patch Set 1: I would prefer that you didn't submit this (1 inline comment) .................................................... File vdsm/libvirtvm.py Line 438: self._preparingMigrationEvt = False I do not understand the motivation for this patch. Do you suggest that if abortJob is called just before migrateToURI, migration would fail? That sounds very odd, to say the least. -- To view, visit http://gerrit.ovirt.org/2533 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0f226e3376b95b64b015885715c0db32eecc5c63 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Shahar Havivi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
