Change in vdsm[master]: Use strings when reporting VM's memory stats

danken at redhat.com danken at redhat.com
Fri Jul 26 18:06:45 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Use strings when reporting VM's memory stats
......................................................................


Use strings when reporting VM's memory stats

This patch changes the API to use strings when reporting memory and
balloon stats for a VM. This fixes an issue with xml-rpc that
can only transfer 32 bit signed numbers.

The maximum memory value when using KiB is about 2TB and we
already hit that.

Change-Id: I60b52fface17e5023c666ebed24dd5fc6b204dbc
Resolves: rhbz#974917
Signed-off-by: Martin Sivak <msivak at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17086
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/vm.py
M vdsm_api/vdsmapi-schema.json
2 files changed, 10 insertions(+), 8 deletions(-)

Approvals:
  Martin Sivák: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I60b52fface17e5023c666ebed24dd5fc6b204dbc
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list