X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=controller.py;h=9979370d5398c92e2f6ea47ec48e2a8bda552594;hb=3b8e438a83ad72d512cb0ec8f5c4c614a4082700;hp=17268d2107e1a18e42a91895c53ef55a43e8635e;hpb=4cbb63bed934afd59fd6131961e2cca9089ed8ce;p=nodemanager.git

diff --git a/controller.py b/controller.py
index 17268d2..9979370 100644
--- a/controller.py
+++ b/controller.py
@@ -1,9 +1,13 @@
-"""Delegate accounts are used to provide secure access to the XMLRPC API.  They are normal Unix accounts with a shell that tunnels XMLRPC requests to the API server."""
+#
+"""Delegate accounts are used to provide secure access to the XMLRPC API.
+They are normal Unix accounts with a shell that tunnels XMLRPC requests to the API server."""
+
+from pwd import getpwnam
+from grp import getgrnam
 
-import accounts
 import logger
 import tools
-from pwd import getpwnam
+import accounts
 
 class Controller(accounts.Account):
     SHELL = '/usr/bin/forward_api_calls'  # tunneling shell
@@ -12,15 +16,16 @@ class Controller(accounts.Account):
     @staticmethod
     def create(name, vref = None):
         add_shell(Controller.SHELL)
-        logger.log_call('/usr/sbin/useradd', '-p', '*', '-s', Controller.SHELL, name)
+        group = getgrnam("slices")[2]
+        logger.log_call(['/usr/sbin/useradd', '-p', '*', '-g', str(group), '-s', Controller.SHELL, name, ])
 
     @staticmethod
-    def destroy(name): logger.log_call('/usr/sbin/userdel', '-r', name)
+    def destroy(name): logger.log_call(['/usr/sbin/userdel', '-r', name, ])
 
     def is_running(self):
-        logger.verbose("Delegate:  %s" % self.name)
+        logger.verbose("controller: is_running:  %s" % self.name)
         return getpwnam(self.name)[6] == self.SHELL
-    
+
 
 def add_shell(shell):
     """Add <shell> to /etc/shells if it's not already there."""