Change in vdsm[master]: domain: use the dd statistics for the domain delay
danken at redhat.com
danken at redhat.com
Mon Jun 3 10:36:06 UTC 2013
Dan Kenigsberg has submitted this change and it was merged.
Change subject: domain: use the dd statistics for the domain delay
......................................................................
domain: use the dd statistics for the domain delay
The previous implementation of getReadDelay was calculating the reading
time (for the domain statistics) checking the execution time of dd.
This patch introduces a low level readspeed command to parse the more
accurate statistics provided by dd in the stderr.
Bug-Url: http://bugzilla.redhat.com/show_bug.cgi?id=948232
Change-Id: Ib3fa54dbbe3475b74bcfe5cbc0c60895818f8b3e
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M tests/miscTests.py
M vdsm/storage/blockSD.py
M vdsm/storage/fileSD.py
M vdsm/storage/misc.py
4 files changed, 63 insertions(+), 11 deletions(-)
Approvals:
Giuseppe Vallarelli: Looks good to me, but someone else must approve
Federico Simoncelli: Verified
Dan Kenigsberg: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/15049
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3fa54dbbe3475b74bcfe5cbc0c60895818f8b3e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list