Roy Golan has submitted this change and it was merged. Change subject: scheduler: Refactor AffinityRulesEnforcementManager ...................................................................... scheduler: Refactor AffinityRulesEnforcementManager * Fix check if a VM is still migrating. * Trigger one migration per cluster instead of one migration per datacenter. * Split unit tests in AffinityRulesEnforcementManager tests and AffinityRulesEnforcementPerCluster tests. * Remove old tests and added more new ones. * Remove not fully implemented features: - lower wakeup frequency when everything is satisfied - temporary suspension of clusters when too many migration failures occur They will be added later. Bug-Url: https://bugzilla.redhat.com/1251742 Change-Id: I8921baa3b5335103d8dc0cd31eed094eb025cba1 Signed-off-by: Roman Mohr <rmohr(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcer.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/MigrationEntryDS.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcerTest.java 5 files changed, 332 insertions(+), 575 deletions(-) Approvals: Roman Mohr: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44804 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8921baa3b5335103d8dc0cd31eed094eb025cba1 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com&gt; Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com&gt; Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com&gt; Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com&gt; Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com&gt; Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org