X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fclient%2Fclient_helper.py;h=809831942035d720f36d4e11bc69f983dde0c4ea;hb=4a9e6751f9f396f463932133b9d62fc925a99ef6;hp=64c8b43a5cb46ae657368387364d43b123ba9b5e;hpb=ce4645a2d3deea2616a2ae3cc0933c6a6b9ec17a;p=sfa.git

diff --git a/sfa/client/client_helper.py b/sfa/client/client_helper.py
index 64c8b43a..80983194 100644
--- a/sfa/client/client_helper.py
+++ b/sfa/client/client_helper.py
@@ -1,37 +1,61 @@
+###
+#
+# Thierry - 2012 sept 21
+#
+# it seems terribly wrong that the client should decide to use PG- or PL- related code
+# esp. in a context where we're trying to have more and more kinds of testbeds involved
+#
+# also, the 'users' field that CreateSliver is expecting (the key point here is to get this right)
+# is specified to have at least a urn and a list of keys, both of these being supported natively
+# in the sfa db
+# So long story short, it seems to me that we should have a common code that fills in 'urn' and 'keys'
+# and then code that tentatively tries to add as much extra info that we can get on these users
+#
+# the fact e.g. that PlanetLab insists on getting a first_name and last_name is not
+# exactly consistent with the GENI spec. of CreateSliver
+#
+
 
 def pg_users_arg(records):
-    users = []  
+    users = []
     for record in records:
-        if record['type'] != 'user': 
+        if record['type'] != 'user':
             continue
-        user = {'urn': record['geni_urn'],
-                'keys': record['keys']}
+        user = {'urn': record['reg-urn'],
+                'keys': record['reg-keys'],
+                'email': record['email']}
         users.append(user)
-    return users    
+    return users
+
 
 def sfa_users_arg(records, slice_record):
     users = []
     for record in records:
-        if record['type'] != 'user': 
+        if record['type'] != 'user':
             continue
-        user = {'urn': record['geni_urn'], #
-                'keys': record['keys'],
-                'email': record['email'], #  needed for MyPLC
-                'person_id': record['person_id'], # needed for MyPLC
-                'first_name': record['first_name'], # needed for MyPLC
-                'last_name': record['last_name'], # needed for MyPLC
-                'slice_record': slice_record, # needed for legacy refresh peer
-                'key_ids': record['key_ids'] # needed for legacy refresh peer
-                }         
+        user = {'urn': record['reg-urn'],
+                'keys': record['reg-keys'],
+                'slice_record': slice_record,
+                }
+        # fill as much stuff as possible from planetlab or similar
+        # note that reg-email is not yet available
+        pl_fields = ['email', 'person_id',
+                     'first_name', 'last_name', 'key_ids']
+        # try to fill all these in
+        for field in pl_fields:
+            if field in record:
+                user[field] = record[field]
         users.append(user)
-    return users        
+
+    return users
+
 
 def sfa_to_pg_users_arg(users):
 
     new_users = []
     fields = ['urn', 'keys']
     for user in users:
-        new_user = dict([item for item in user.items() \
-          if item[0] in fields])
+        new_user = dict([item for item in list(user.items())
+                         if item[0] in fields])
         new_users.append(new_user)
-    return new_users        
+    return new_users