Roy Golan has submitted this change and it was merged.
Change subject: core: fix high memory limit default value
......................................................................
core: fix high memory limit default value
In the secondary methods of power saving load balancing
for finding source/destination hosts by memory utilization
the default value of high memory limit is 0.
In case no value was inserted to this limit -
the scheduling policy will filter out all hosts since no host
will satisfy host available memory < 0.
fix - the default value was changed to max long
in case no value was inserted
Change-Id: I200f1886a0de9c1ff1d15f863d41af67cce07119
Bug-Url:
https://bugzilla.redhat.com/1354281
Signed-off-by: yquinn <yquinn(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingBalancePolicyUnit.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingBalancePolicyUnitTest.java
2 files changed, 55 insertions(+), 2 deletions(-)
Approvals:
Martin Sivák: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
Yanir Quinn: Verified
--
To view, visit
https://gerrit.ovirt.org/60668
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I200f1886a0de9c1ff1d15f863d41af67cce07119
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>