Arik Hadas has submitted this change and it was merged.
Change subject: core: fix check for whether vm is starting up
......................................................................
core: fix check for whether vm is starting up
Previously the check relied on the result of getLastVdsRunOn to see if
the VM is starting up since preferred hosts are supposed to be used
only during VM startup. But VmDynamic#lastVdsRunOn never contains
the real host that the VM ran on when getting to this method (it is
always null so this check is passed also for migrating VMs).
Therefore, this check is changed.
Change-Id: I02832a76d94f7a02d6f2b82970461d14a628ed74
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PreferredHostsWeightPolicyUnit.java
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Martin Sivák: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit
https://gerrit.ovirt.org/65282
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I02832a76d94f7a02d6f2b82970461d14a628ed74
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>