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

[rgolan at redhat.com]

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 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, 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 at redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: automation at ovirt.org