Shu Ming has posted comments on this change.
Change subject: add "uptime" field to the getStats (getVdsStats in vdsClient)
call
......................................................................
Patch Set 2: I would prefer that you didn't submit this
(1 inline comment)
....................................................
File vdsm/API.py
Line 1102: stats['memShared'] = self._memShared() / Mbytes
Line 1103:
Line 1104: stats['netConfigDirty'] = str(self._cif._netConfigDirty)
Line 1105: stats['generationID'] = self._cif._generationID
Line 1106: stats['uptime'] = time.time() - STARTTIME
I don't like STARTTIME to be a global one. Would it better to store it in some
other places like self._cif.starttime?
Line 1107: return {'status': doneCode, 'info': stats}
Line 1108:
Line 1109: def setLogLevel(self, level):
Line 1110: """
--
To view, visit http://gerrit.ovirt.org/8008
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc998117e25faff17c2f3f6d66e2a961fc93e029
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Gal Hammer <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Peter V. Saveliev <[email protected]>
Gerrit-Reviewer: Ryan Harper <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
