From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: vm: migration: handle destroy on startup on dest
......................................................................


vm: migration: handle destroy on startup on dest

When a pre-copy migration is not converging, the
source Vdsm may decide to abort the operation, and
during this process it destroys the destination side Vm.

This is part of the normal flow of operations, and one
of the expected and supported outcomes of the pre-copy
migrations.

What should not happen, however, is that the destination
side VM reports error in the logs. This is wrong for users,
which see a scary and pointless stacktrace, and may
confuse Engine if it sees a VM down in Error.

With this patch, we add explicit check for this condition,
and we let the destination VM to be down without error.

See
http://lists.ovirt.org/pipermail/users/2016-December/044477.html
For a recent case.

Change-Id: I9fa5d99ca10877bb2c230f892562c5ebee455e21
Bug-Url: https://bugzilla.redhat.com/1134974
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified
  Milan Zamazal: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/67858
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9fa5d99ca10877bb2c230f892562c5ebee455e21
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to