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 <ykap...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/39202 Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-by: Francesco Romani <from...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Feng Yang <yangf...@cloud-times.com> Reviewed-by: Dan Kenigsberg <dan...@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 <ykap...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Feng Yang <yangf...@cloud-times.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches