Change in ovirt-engine[master]: core: fix high memory limit default value

rgolan at redhat.com rgolan at redhat.com
Mon Aug 1 09:35:40 UTC 2016


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 at 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 at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list