Omer Frenkel has submitted this change and it was merged.
Change subject: core: use db status when processing down vms
......................................................................
core: use db status when processing down vms
when analyzing vm in down status, if the vm is not running on that host,
the status from db is not taken in consideration.
in some cases, like during migration, the analyzer can use the status
from db, even if the vm is not running this host, in order to understand
the current state of the vm.
scenrio for example: vm moves to down on destination, since the analyzer
dont use the status from the db, it doesnt know its migration process
and clears the migrate command from the cache, breaking rerun/further
treatment of the command.
Change-Id: I1a893f7e06e417ca1a516cb04feb3a5defdb9c23
Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java
1 file changed, 11 insertions(+), 9 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Omer Frenkel: Verified
Arik Hadas: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/43904
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1a893f7e06e417ca1a516cb04feb3a5defdb9c23
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org