Dan Kenigsberg has submitted this change and it was merged.

Change subject: Fix ballooning behaviour of freshly received VM
......................................................................


Fix ballooning behaviour of freshly received VM

There is an issue with freshly received VMs. MOM might ask for
the current balloon status before the stats threads collect all
data. The target will be reported as 0 in that case and the VM
will be requested to return all its free memory.

The balloon info method won't return any data before it
has everything ready after this fix is applied.

This also fixes the unittest that was using balloon for checking
whether the stats system works.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1123274
Change-Id: If44c8e9d65b7221b2dec5fdd1f2d7d4343f7aeaa
Signed-off-by: Martin Sivak <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/34887
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Adam Litke <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M tests/vmTests.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/virt/vm.py
3 files changed, 25 insertions(+), 20 deletions(-)

Approvals:
  Adam Litke: Looks good to me, but someone else must approve
  Martin Sivák: Verified
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If44c8e9d65b7221b2dec5fdd1f2d7d4343f7aeaa
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to