Change in vdsm[master]: virt: stats: split up stats production

danken at redhat.com danken at redhat.com
Tue Mar 31 14:46:38 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: virt: stats: split up stats production
......................................................................


virt: stats: split up stats production

VDSM needs to translate the stats it gathered -mostly
from libvirt- in the format established in the
VDSM schema, which is the format the consumers
of its API (read: Engine) expect.

To do so, we used to have a bunch of methods in
the VmStatsThread class. But such methods actually
used few if any at all of the class state, so
it is better to make them simple functions.

This is what this patch does.

Change-Id: I1a1d42b10714fa69c78f58ffaab7f7a32aed47ba
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/36730
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M debian/vdsm.install
M tests/vmTests.py
M vdsm.spec.in
M vdsm/virt/Makefile.am
A vdsm/virt/vmstats.py
5 files changed, 296 insertions(+), 18 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit https://gerrit.ovirt.org/36730
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a1d42b10714fa69c78f58ffaab7f7a32aed47ba
Gerrit-PatchSet: 45
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list