Arik Hadas has submitted this change and it was merged. Change subject: core: remove handleVmOnDown ...................................................................... core: remove handleVmOnDown The handling of report of Down VM from host that is different than the one we expect the VM to run on in handleVmDown is incorrect - we neither want it to remove the async command nor to update the dynamic VM data. The handling of report of Down VM from the host that we expect the VM to run on was merged to proceedDownVm, rednering handleVmOnDown redundant and therefore is removed. Change-Id: Ie2d7b1994c3066e6964f1addab4a659d2004add4 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, 22 insertions(+), 34 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/55736 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2d7b1994c3066e6964f1addab4a659d2004add4 Gerrit-PatchSet: 9 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: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>