Change in ovirt-engine[ovirt-engine-3.4]: core: update the committed memory properly

oourfali at redhat.com oourfali at redhat.com
Thu Mar 20 22:28:03 UTC 2014


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/1078553
Signed-off-by: Arik Hadas <ahadas at 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/25953
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I04be4ca80456ffbf47af2f883a33672166c25cb6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list