Yaniv Bronhaim has posted comments on this change.
Change subject: isRunning didn't check local variable before reading saved data
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File vdsm/supervdsm.py
Line 144: self._firstLaunch = True
Line 145:
Line 146: def isRunning(self):
Line 147: if self._firstLaunch or self._svdsm is None:
Line 148: return False
Although firstLaunch is quite redundant at all, but i don't want to include omitting
it as part of this patch. we use it only to distinguish the case that svdsm.pid or
timestamp file was removed manually (line 158) , in that case we raise exception.
But still, it helps to know when we need to restart svdsm without reading the internal
files..
Line 149:
Line 150: try:
Line 151: with open(self.pidfile, "r") as f:
Line 152: spid = f.read().strip()
--
To view, visit
http://gerrit.ovirt.org/10491
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9dec0c6955dadcd959cc1c8df4e9745322fb0ce3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Royce Lv <lvroyce(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server