Change in vdsm[master]: Using pidStat to get pid creation time

ybronhei at redhat.com ybronhei at redhat.com
Thu May 23 12:56:50 UTC 2013


Yaniv Bronhaim has posted comments on this change.

Change subject: Using pidStat to get pid creation time
......................................................................


Patch Set 1: Verified

(1 inline comment)

....................................................
File vdsm/supervdsm.py
Line 166:             else:
Line 167:                 raise
Line 168: 
Line 169:         try:
Line 170:             pTime = str(utils.pidStat(int(spid))[21])
after merging http://gerrit.ovirt.org/#/c/11033 I'll change it to utils.pidStat((int(spid)).starttime ... sorry for the explicit number
Line 171:         except OSError as e:
Line 172:             if e.errno == ESRCH:
Line 173:                 # Means pid is not exist, svdsm was killed
Line 174:                 return False


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie8e056472e5c83104168e55a8dd01a941a2357f3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list