Change in vdsm[master]: Supervdsm didn't create pid file when starting up

ybronhei at redhat.com ybronhei at redhat.com
Fri May 31 11:13:32 UTC 2013


Yaniv Bronhaim has posted comments on this change.

Change subject: Supervdsm didn't create pid file when starting up
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/supervdsmServer.py
Line 360:             sys.exit(errno.EPERM)
Line 361: 
Line 362:         pidfile = P_VDSM_RUN + 'supervdsmd.pid'
Line 363:         pid = str(os.getpid())
Line 364:         file(pidfile, 'w').write(pid + "\n")
thanks.. if I fix it here, on the way I'll also fix it where I took it from :)
Line 365: 
Line 366:         log.debug("Parsing cmd args")
Line 367:         address = sys.argv[1]
Line 368: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I649179ca1c614dfcff91da9fada785c57303f57c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list