Change in ovirt-engine[master]: core: update the committed memory properly
Arik Hadas 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/1078553
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/25943
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I04be4ca80456ffbf47af2f883a33672166c25cb6
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server