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