Hello Germano Veit Michel, Francesco Romani,
I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/47105
to review the following change.
Change subject: vdsm-tool: Do not panic with sanlock empty groups
......................................................................
vdsm-tool: Do not panic with sanlock empty groups
This fixes the following error:
File "/usr/lib/python2.6/site-packages/vdsm/tool/configurators/
sanlock.py", line 87, in isconfigured
.strip().split(" ")]
ValueError: invalid literal for int() with base 10: ''
Which happens when sanlock "Groups:" line is empty:
$ grep "Groups:" /proc/<sanlock pid>/status
Groups:
Sanlock must have its groups properly setup, but we should not
blow up because of this.
Fixed by using split() instead of split(' '), which cannot handle empty
string and is the wrong way to split in most cases.
Change-Id: I47f080beb748353970dec0753c9a0a7b1dd09bc6
Bug-Url:https://bugzilla.redhat.com/1267444
Backport-To: 3.6
Backport-To: 3.5
Signed-off-by: Germano Veit Michel <germano(a)redhat.com>
Signed-off-by: Nir Soffer <nsoffer(a)redhat.com>
Reviewed-on:
https://gerrit.ovirt.org/46742
Reviewed-by: Francesco Romani <fromani(a)redhat.com>
Continuous-Integration: Dan Kenigsberg <danken(a)redhat.com>
---
M lib/vdsm/tool/configurators/sanlock.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/05/47105/1
diff --git a/lib/vdsm/tool/configurators/sanlock.py
b/lib/vdsm/tool/configurators/sanlock.py
index 1c2b3dd..5f21b46 100644
--- a/lib/vdsm/tool/configurators/sanlock.py
+++ b/lib/vdsm/tool/configurators/sanlock.py
@@ -84,7 +84,7 @@
if status_line.startswith(proc_status_group_prefix):
groups = [int(x) for x in status_line[
len(proc_status_group_prefix):]
- .strip().split(" ")]
+ .strip().split()]
break
else:
raise InvalidConfig(
--
To view, visit
https://gerrit.ovirt.org/47105
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I47f080beb748353970dec0753c9a0a7b1dd09bc6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Germano Veit Michel <germano(a)redhat.com>