Roy Golan 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@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java 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 3 files changed, 69 insertions(+), 9 deletions(-) Approvals: Roman Mohr: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42633 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9908810a7f373a8098a6765e1c8f95e53af1a0a Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban@redhat.com> Gerrit-Reviewer: Dudi Maroshi <dudi@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban@redhat.com> Gerrit-Reviewer: automation@ovirt.org