Federico Simoncelli has uploaded a new change for review.
Change subject: vdsm-tool: handle missing sanlock pid file
......................................................................
vdsm-tool: handle missing sanlock pid file
Change-Id: I711d9b413415ca9372520e6bb25cf6d39ff571a3
Signed-off-by: Federico Simoncelli <fsimo...@redhat.com>
---
M lib/vdsm/tool/sanlock.py
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/12/19112/1
diff --git a/lib/vdsm/tool/sanlock.py b/lib/vdsm/tool/sanlock.py
index c64c874..650a5cf 100644
--- a/lib/vdsm/tool/sanlock.py
+++ b/lib/vdsm/tool/sanlock.py
@@ -38,13 +38,13 @@
supplementary groups.
"""
- sanlock_pid = open(SANLOCK_PID, "r").readline().strip()
-
try:
+ sanlock_pid = open(SANLOCK_PID, "r").readline().strip()
sanlock_status = open(PROC_STATUS_PATH % sanlock_pid, "r")
except IOError as e:
if e.errno == os.errno.ENOENT:
return 0 # service is not running, returning
+ raise
for status_line in sanlock_status:
if status_line.startswith(PROC_STATUS_GROUPS):
--
To view, visit http://gerrit.ovirt.org/19112
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I711d9b413415ca9372520e6bb25cf6d39ff571a3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
