Change in vdsm[master]: sampling: hoststats: switch to SampleWindow

danken at redhat.com danken at redhat.com
Tue Jun 30 19:23:06 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: sampling: hoststats: switch to SampleWindow
......................................................................


sampling: hoststats: switch to SampleWindow

HostStatsThread has its own sampling storage code,
like VmStatsThread had before the introduction of
the SampleWindow class.

The API of SampleWindow covers the HostStatsThread needs,
so this patch ports HostStatsThread to SampleWindow,
to reduce duplication.

Change-Id: I5ec5712ee1fb69ed3d8dbc28ca4e6511998270c9
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40428
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/samplingTests.py
M vdsm/virt/sampling.py
2 files changed, 10 insertions(+), 12 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5ec5712ee1fb69ed3d8dbc28ca4e6511998270c9
Gerrit-PatchSet: 16
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: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list