Change in vdsm[master]: configurator: use sanlock user constant

dougsland at redhat.com dougsland at redhat.com
Tue Mar 25 14:37:25 UTC 2014


Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: configurator: use sanlock user constant
......................................................................

configurator: use sanlock user constant

Replace the static value with a constant.

Change-Id: I43464961cba6b6b761a98ad4b72a271228914ca1
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M lib/vdsm/constants.py.in
M lib/vdsm/tool/configurator.py
2 files changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/26080/1

diff --git a/lib/vdsm/constants.py.in b/lib/vdsm/constants.py.in
index 6653d54..4ddfe84 100644
--- a/lib/vdsm/constants.py.in
+++ b/lib/vdsm/constants.py.in
@@ -41,6 +41,9 @@
 QEMU_PROCESS_USER = '@QEMUUSER@'
 QEMU_PROCESS_GROUP = '@QEMUGROUP@'
 
+# Sanlock definitions
+SANLOCK_USER = '@SNLKUSER@'
+
 #
 # The username of SASL authenticating for libvirt connection
 #
diff --git a/lib/vdsm/tool/configurator.py b/lib/vdsm/tool/configurator.py
index d1c876c..896aa25 100644
--- a/lib/vdsm/tool/configurator.py
+++ b/lib/vdsm/tool/configurator.py
@@ -24,7 +24,8 @@
 
 from .. import utils
 from . import service, expose
-from ..constants import P_VDSM_EXEC, QEMU_PROCESS_GROUP, VDSM_GROUP
+from ..constants import P_VDSM_EXEC, QEMU_PROCESS_GROUP, \
+        SANLOCK_USER, VDSM_GROUP
 
 
 class _ModuleConfigure(object):
@@ -132,7 +133,7 @@
                 '-a',
                 '-G',
                 ','.join(self.SANLOCK_GROUPS),
-                'sanlock'
+                SANLOCK_USER
             ),
             raw=True,
         )


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I43464961cba6b6b761a98ad4b72a271228914ca1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list