From Arik Hadas <ahadas@redhat.com>:

Arik Hadas has submitted this change and it was merged. Change subject: core: prevent NPE on unmanaged migrations ...................................................................... core: prevent NPE on unmanaged migrations During migrations that were not initiated by the engine we do not have the destination host, unfortunately. This patch prevents the NPE that is then thrown in the hand-over by a simple null-check. The VM would then be set to MigratingTo and its runOnVds would be null. Thus, we enhance the VM analysing with a proper handling for this new state - when the VM would be detected on the destination host, its runOnVds will change to point to that host. Change-Id: Ie11c0eb14318d8ab28f613cbf012879aba4f2ab9 Bug-Url: https://bugzilla.redhat.com/1392903 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 14 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Shmuel Leib Melamud: Looks good to me, but someone else must approve Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/66657 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie11c0eb14318d8ab28f613cbf012879aba4f2ab9 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>