Changeset f3898f7 for fedd/federation/emulab_access.py

Timestamp:

Oct 5, 2011 5:11:01 PM (13 years ago)

Author:

Ted Faber <faber@…>

Branches:

compt_changes, info-ops, master

Children:

b6a6206

Parents:

53b5c18

git-author:

Ted Faber <faber@…> (10/05/11 17:10:01)

git-committer:

Ted Faber <faber@…> (10/05/11 17:11:01)

Message:

Simple support for experiments in groups. Closes #32

File:

• fedd/federation/emulab_access.py (modified) (8 diffs)

fedd/federation/emulab_access.py

@@ -937 +937 @@
         vchars_re = '[^' + string.ascii_letters + string.digits  + '-]'
 
+        self.state_lock.acquire()
+        if aid in self.allocation:
+            proj = self.allocation[aid].get('project', None)
+            if not proj: 
+                proj = self.allocation[aid].get('sproject', None)
+        self.state_lock.release()
+
+        if not proj:
+            raise service_error(service_error.internal, 
+                    "Can't find project for %s" %aid)
+
         for a in attrs:
             if a['attribute'] in configs:
@@ -954 +965 @@
                 ename = a['value']
 
-        # Names longer than the emulab max are discarder
-        if ename and len(ename) <= self.max_name_len:
+        # Names longer than the emulab max are discarded
+        # Projects with a group require nonce experiment names as well
+        if ename and len(ename) <= self.max_name_len and '/' not in proj:
             # Clean up the experiment name so that emulab will accept it.
             ename = re.sub(vchars_re, '-', ename)
@@ -976 +988 @@
         self.state_lock.acquire()
         if aid in self.allocation:
-            proj = self.allocation[aid].get('project', None)
-            if not proj: 
-                proj = self.allocation[aid].get('sproject', None)
             user = self.allocation[aid].get('user', None)
             self.allocation[aid]['experiment'] = ename
@@ -995 +1004 @@
             self.write_state()
         self.state_lock.release()
-
-        if not proj:
-            raise service_error(service_error.internal, 
-                    "Can't find project for %s" %aid)
 
         if not user:
@@ -1095 +1100 @@
                             certfile, tmpdir)
 
+            if '/' in proj: proj, gid = proj.split('/')
+            else: gid = None
+
+
             # Set up userconf and seer if needed
             self.configure_userconf(services, tmpdir)
@@ -1112 +1121 @@
                     debug=self.create_debug, log=alloc_log, boss=self.boss,
                     cert=self.xmlrpc_cert)
-            rv = starter(self, ename, proj, user, expfile, tmpdir)
+            rv = starter(self, ename, proj, user, expfile, tmpdir, gid=gid)
         except service_error, e:
             err = e
@@ -1164 +1173 @@
             raise service_error(service_error.internal, 
                     "Can't find project for %s" % aid)
+        else:
+            if '/' in proj: proj, gid = proj.split('/')
+            else: gid = None
 
         if not user:
@@ -1173 +1185 @@
         stopper = self.stop_segment(keyfile=self.ssh_privkey_file,
                 debug=self.create_debug, boss=self.boss, cert=self.xmlrpc_cert)
-        stopper(self, user, proj, ename)
+        stopper(self, user, proj, ename, gid)
         return { 'allocID': req['allocID'], 'proof': proof.to_dict() }