Change in vdsm[master]: migration: remove getVmStats before migration
danken at redhat.com
danken at redhat.com
Fri May 15 10:56:55 UTC 2015
Dan Kenigsberg has submitted this change and it was merged.
Change subject: migration: remove getVmStats before migration
......................................................................
migration: remove getVmStats before migration
Calling getVmStats before migration to check if the
same vm exists on the destination host is redundant.
If it does, then our system is in much bigger trouble
and migration will fail anyway.
and if the connection was not established we will fail
in the next step of the migration anyway.
Change-Id: Ic83ab46173d5d7e5bfd85bf00b8f86a530da5dd4
Signed-off-by: Yeela Kaplan <ykaplan at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/39202
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Feng Yang <yangfeng at cloud-times.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/virt/migration.py
1 file changed, 0 insertions(+), 9 deletions(-)
Approvals:
Piotr Kliczewski: Looks good to me, but someone else must approve
Yeela Kaplan: Verified
Yaniv Bronhaim: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Feng Yang: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/39202
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic83ab46173d5d7e5bfd85bf00b8f86a530da5dd4
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Feng Yang <yangfeng at cloud-times.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the vdsm-patches
mailing list