Oved Ourfali has submitted this change and it was merged.
Change subject: core: update the committed memory properly
......................................................................
core: update the committed memory properly
The computation of the committed memory was wrong because it included
VMs which are down as well. The problem was that we were iterating the
wrong collections of VMs, we should iterate over the VMs which we got
back from VDSM and take only them into consideration.
Change-Id: I04be4ca80456ffbf47af2f883a33672166c25cb6
Bug-Url:
https://bugzilla.redhat.com/1074478
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
1 file changed, 13 insertions(+), 6 deletions(-)
Approvals:
Omer Frenkel: Looks good to me, approved
Arik Hadas: Verified
--
To view, visit
http://gerrit.ovirt.org/25948
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I04be4ca80456ffbf47af2f883a33672166c25cb6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server