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