Change in ovirt-engine[master]: core: remove handleVmOnDown

ahadas at redhat.com ahadas at redhat.com
Mon Apr 18 08:52:53 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list