Gilad Chaplik has uploaded a new change for review.

Change subject: core: Max memory for scheduling updates wrongly
......................................................................

core: Max memory for scheduling updates wrongly

The calc should include all running VMs.
VmsStatisticsFetcher includes all VMs and iterates
when getRefreshStats() == true.

Change-Id: I87f0f51f75a54100a54d66ba30d34a903d3887bb
Bug-Url: https://bugzilla.redhat.com/1182007
Signed-off-by: Gilad Chaplik <gchap...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/70/36970/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
index 73484ca..f0673c5 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
@@ -737,6 +737,9 @@
      *                  getVdsCaps
      */
     void refreshCommitedMemory() {
+        if (!vdsManager.getRefreshStatistics()) {
+            return;
+        }
         int memCommited = vds.getGuestOverhead();
         int vmsCoresCount = 0;
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I87f0f51f75a54100a54d66ba30d34a903d3887bb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to