Change in ovirt-engine[master]: core: add max memory for scheduling VMs as a calculated fiel...

rgolan at redhat.com rgolan at redhat.com
Thu Sep 13 08:10:57 UTC 2012


Roy Golan has submitted this change and it was merged.

Change subject: core: add max memory for scheduling VMs as a calculated field to VDS
......................................................................


core: add max memory for scheduling VMs as a calculated field to VDS

This new calculated field is mostly to keep the user informed of the
max amount of memory is free to schedule a new VM. The number reflects
the absolute max which the VM scheduler can't exceed and is subjected to
future scheduling policies, which means the actual number can be lower.

The calculation is:

     (getmax_vds_memory_over_commit() * getphysical_mem_mb() / 100.0f) -

      (getmem_commited() + getreserved_mem())

Field is added to GUI as Mb and to REST as bytes (traditionally for all
memory fields there)

Change-Id: I8cb338f79682bdc550c4cc6e0667ba1fea50a9c6
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostMapperTest.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostGeneralModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGeneralView.java
8 files changed, 52 insertions(+), 2 deletions(-)

Approvals:
  Roy Golan: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7825
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8cb338f79682bdc550c4cc6e0667ba1fea50a9c6
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list