Milan Zamazal has posted comments on this change. Change subject: migration: wait properly for migration to begin ......................................................................
Patch Set 2: (4 comments) https://gerrit.ovirt.org/#/c/60073/2//COMMIT_MSG Commit Message: PS2, Line 13: even the ... even though the ... ? PS2, Line 17: presnet ... present PS2, Line 18: it's ... its https://gerrit.ovirt.org/#/c/60073/2/vdsm/virt/migration.py File vdsm/virt/migration.py: Line 705: Line 706: if self._stop.isSet(): Line 707: break Line 708: Line 709: if progress.ongoing: Is this condition still necessary? Line 710: self.progress = progress Line 711: self._vm.log.info('%s', progress) Line 712: Line 713: def stop(self): -- To view, visit https://gerrit.ovirt.org/60073 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If1ba369a0992c2473acf1395fad0b0c260c250ba Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
