From Tal Nisan <tnisan(a)redhat.com>:
Tal Nisan has submitted this change and it was merged.
Change subject: core: remove unneeded query when getting vms to move to unknown
......................................................................
core: remove unneeded query when getting vms to move to unknown
When host monitoring gets a network exception during monitoring of a
specific host, it switches this host to maintenance and moves all the
VMs running on this host to Unknown.
The retrieval of VMs that need to be switched to unknown was composed
of two parts:
1. get all the VMs running on the host (their run_on_vds points to
that host).
2. get all the VMs that migrate to the host (their migrating_to_vds
points to that host) and also are in MigratingTo status.
But (2) is redundant since at this phase of the migration run_on_vds
must point to the destination host. Therefore, this part is removed.
Note: the query for VM dynamics from the database was moved as a
preparation for future changes.
Change-Id: I1727da9d56cf3b21464da913942f115abeb1bd5b
Bug-Url:
https://bugzilla.redhat.com/1317429
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
1 file changed, 7 insertions(+), 16 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit
https://gerrit.ovirt.org/69251
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1727da9d56cf3b21464da913942f115abeb1bd5b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>