Change in ovirt-engine[master]: scheduler: Refactor AffinityRulesEnforcementManager
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>
---
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>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org