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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to