Change in ovirt-engine[ovirt-engine-3.6]: scheduler: Refactor AffinityRulesEnforcementManager

[tnisan at redhat.com]

Tal Nisan 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 at redhat.com>
---
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, 329 insertions(+), 574 deletions(-)

Approvals:
  Tal Nisan: Passed CI tests
  Roman Mohr: Verified
  Roy Golan: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8921baa3b5335103d8dc0cd31eed094eb025cba1
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Jenkins CI
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: automation at ovirt.org