Change in vdsm[master]: stats: move the lastCheck computation to getRepoStats

danken at redhat.com danken at redhat.com
Tue Oct 23 09:08:24 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: stats: move the lastCheck computation to getRepoStats
......................................................................


stats: move the lastCheck computation to getRepoStats

Since getVdsStats relies on repoStats for the storage domain statistics
it makes sense to move the lastCheck computation into the latter one.
Moreover the repoStats call before this patch was returning the unix
timestamp for the lastCheck (wrong) instead of the elapsed time.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=868721
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Change-Id: I58de0c517bc6940f21c7f6506854c9d9735e964b
---
M vdsm/storage/sp.py
M vdsm/utils.py
2 files changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Federico Simoncelli: Verified
  Dan Kenigsberg: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I58de0c517bc6940f21c7f6506854c9d9735e964b
Gerrit-PatchSet: 4
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: Ryan Harper <ryanh at us.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list