Change in ovirt-engine[ovirt-engine-3.6]: core: Added sorting to AREM

tnisan at redhat.com tnisan at redhat.com
Thu Sep 17 08:12:22 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: Added sorting to AREM
......................................................................


core: Added sorting to AREM

- Affinity Rules Enforcement manager need to make sure that affinity
groups are sorted first by affinity group's size (From largest to
smallest) and then by first VM's Guid. That's why we use GuidComparator
to sort the affinity groups.

- Added unit tests to check correct sorting.

Change-Id: Id9908810a7f373a8098a6765e1c8f95e53af1a0a
Bug-Url: https://bugzilla.redhat.com/1254627
Signed-off-by: Tomer Saban <tsaban at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcer.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcerTest.java
2 files changed, 59 insertions(+), 5 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Tomer Saban: Verified
  Roy Golan: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/46208
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id9908810a7f373a8098a6765e1c8f95e53af1a0a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list