Francesco Romani has submitted this change and it was merged.

Change subject: migration: make MonitorThread own DowntimeThread
......................................................................


migration: make MonitorThread own DowntimeThread

A future patch wants to tune the legacy downtime thread lifecycle
to work more efficiently with QEMU iterations during migrations.
To make this easier, we make monitorThread own the downtimeThread
lifecycle.

Change-Id: I3ed5a95dcd8576d0391df175f43cd4677b24e57a
Bug-Url: https://bugzilla.redhat.com/1339521
Backport-To: 3.6
Signed-off-by: Francesco Romani <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/56685
Reviewed-by: Martin Polednik <[email protected]>
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/57933
Reviewed-by: Milan Zamazal <[email protected]>
Reviewed-by: Tomas Jelinek <[email protected]>
---
M vdsm/virt/migration.py
1 file changed, 26 insertions(+), 7 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified; Looks good to me, approved
  Martin Polednik: Looks good to me, but someone else must approve
  Milan Zamazal: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/57933
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3ed5a95dcd8576d0391df175f43cd4677b24e57a
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Milan Zamazal <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to