Change in ovirt-engine[master]: core: Change how Affinity behaves if the assumptions are inv...

[dfediuck at redhat.com]

Doron Fediuck has submitted this change and it was merged.

Change subject: core: Change how Affinity behaves if the assumptions are invalid
......................................................................


core: Change how Affinity behaves if the assumptions are invalid

This fixes the situation when VMs belonging to the same hard
constraint positive affinity group are actually running on
more than one host.

It used to fail the scheduling task for any VM belonging to
the same affinity group (both new VMs and migrating ones).

This patch just passes all hosts with VM from that group
to the weighting mechanism making it possible for the
balancer or administrator to recover from this situation.

This change also makes it easier to report per host scheduling
information which we need for the following patches.

Change-Id: I506bcb6e96c622694e6bf7b8ce7a0f54a82e5713
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1080521
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/VmAffinityFilterPolicyUnit.java
1 file changed, 43 insertions(+), 48 deletions(-)

Approvals:
  Martin Sivák: Verified
  Jiří Moskovčák: Looks good to me, but someone else must approve
  Doron Fediuck: Looks good to me, approved
  Kobi Ianko: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/26619
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I506bcb6e96c622694e6bf7b8ce7a0f54a82e5713
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Kobi Ianko <kobi at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server