Change in ovirt-engine[master]: core: fix check for whether vm is starting up

[ahadas at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>