fix typo

[sfa.git] / sfa / methods / register.py

diff --git a/sfa/methods/register.py b/sfa/methods/register.py
index 9f5d019..a27cedf 100644 (file)
--- a/sfa/methods/register.py
+++ b/sfa/methods/register.py
@@ -44,13 +44,17 @@ class register(Method):
         auth_name = self.api.auth.get_authority(name)
         auth_info = self.api.auth.get_auth_info(auth_name)
         table = self.api.auth.get_auth_table(auth_name)
-        
+        pub_key = None  
         # make sure record has a gid
         if 'gid' not in record:
             uuid = create_uuid()
             pkey = Keypair(create=True)
-            if 'keys' in record and record['keys']:
-                pkey = convert_public_key(record['keys'][0])
+            if 'key' in record and record['key']:
+                if isinstance(record['key'], list):
+                    pub_key = record['key'][0]
+                else:
+                    pub_key = record['key']
+                pkey = convert_public_key(pub_key)
             
             gid_object = self.api.auth.hierarchy.create_gid(name, uuid, pkey)
             gid = gid_object.save_to_string(save_parents=True)
@@ -62,7 +66,7 @@ class register(Method):
         if existing_records:
             raise ExistingRecord(name)
         
-        if (type == "sa") or (type=="ma"):
+        if type in ["authority"]:
             # update the tree
             if not self.api.auth.hierarchy.auth_exists(name):
                 self.api.auth.hierarchy.create_auth(name)
@@ -76,30 +80,31 @@ class register(Method):
             gid = auth_info.get_gid_object()
             record.set_gid(gid.save_to_string(save_parents=True))
 
-            # if registering a sa, see if a ma already exists
-            # if registering a ma, see if a sa already exists
-            if (type == "sa"):
-                other_rec = table.resolve("ma", record.get_name())
-            elif (type == "ma"):
-                other_rec = table.resolve("sa", record.get_name())
-
-            if other_rec:
-                print >> log, "linking ma and sa to the same plc site"
-                pointer = other_rec[0].get_pointer()
-            else:
-                pl_record = self.api.geni_fields_to_pl_fields(type, name, record)
-                print >> log, "adding site with fields", pl_record
+            pl_record = self.api.geni_fields_to_pl_fields(type, name, record)
+            sites = self.api.plshell.GetSites(self.api.plauth, [pl_record['login_base']])
+            if not sites:    
                 pointer = self.api.plshell.AddSite(self.api.plauth, pl_record)
+            else:
+                pointer = sites[0]['site_id']
 
             record.set_pointer(pointer)
 
         elif (type == "slice"):
             pl_record = self.api.geni_fields_to_pl_fields(type, name, record)
-            pointer = self.api.plshell.AddSlice(self.api.plauth, pl_record)
+            slices = self.api.plshell.GetSlices(self.api.plauth, [pl_record['name']])
+            if not slices: 
+                pointer = self.api.plshell.AddSlice(self.api.plauth, pl_record)
+            else:
+                pointer = slices[0]['slice_id']
             record.set_pointer(pointer)
 
         elif (type == "user"):
-            pointer = self.api.plshell.AddPerson(self.api.plauth, dict(record))
+            persons = self.api.plshell.GetPersons(self.api.plauth, [record['email']])
+            if not persons:
+                pointer = self.api.plshell.AddPerson(self.api.plauth, dict(record))
+            else:
+                pointer = persons[0]['person_id']
+ 
             if 'enabled' in record and record['enabled']:
                 self.api.plshell.UpdatePerson(self.api.plauth, pointer, {'enabled': record['enabled']})
             login_base = get_leaf(auth_info.hrn)
@@ -109,19 +114,27 @@ class register(Method):
             record.set_pointer(pointer)
            
            # Add the user's key
-            if record['keys']:
-               self.api.plshell.AddPersonKey(self.api.plauth, pointer, {'key_type' : 'ssh', 'key' : record['keys'][0]})
+            if pub_key:
+                self.api.plshell.AddPersonKey(self.api.plauth, pointer, {'key_type' : 'ssh', 'key' : pub_key})
 
         elif (type == "node"):
             pl_record = self.api.geni_fields_to_pl_fields(type, name, record)
             login_base = hrn_to_pl_login_base(auth_name)
-            pointer = self.api.plshell.AddNode(self.api.plauth, login_base, pl_record)
+            nodes = self.api.plshell.GetNodes(self.api.plauth, [pl_record['hostname']])
+            if not nodes:
+                pointer = self.api.plshell.AddNode(self.api.plauth, login_base, pl_record)
+            else:
+                pointer = nodes[0]['node_id']
             record.set_pointer(pointer)
 
         else:
             raise UnknownGeniType(type)
 
-        table.insert(record)
+        # SFA upcalls may exist in PLCAPI and they could have already added the
+        # record for us. Lets check if the record already exists  
+        existing_records = table.resolve(type, name)
+        if not existing_records:
+            table.insert(record)
 
         # update membership for researchers, pis, owners, operators
         self.api.update_membership(None, record)