Change in vdsm[ovirt-3.4]: virt: Capture migration start time after the semaphore was a...

ybronhei at redhat.com ybronhei at redhat.com
Wed May 14 10:18:24 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: virt: Capture migration start time after the semaphore was accquired
......................................................................


virt: Capture migration start time after the semaphore was accquired

Previously the migration start time was captured at the start of the
MigrationSourceThread which leads to aborting migrations when the VMs
had too long to wait for the accquisition of the migration semaphore.

This patch captures the start time in the moment the migration really
begins. Additionally this patch logs the time the migration took to
accquire the migration semaphore to improve debugging of the logs.

Change-Id: I209023f5f4a3ccee5db4d43b17627b09ca55bef8
Bug-Url: https://bugzilla.redhat.com/1097298
Signed-off-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/27135
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Michal Skrivanek <michal.skrivanek at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/27637
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/vm.py
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Vinzenz Feenstra: Verified
  Francesco Romani: Looks good to me, but someone else must approve
  Michal Skrivanek: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/27637
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I209023f5f4a3ccee5db4d43b17627b09ca55bef8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list