fix ldap bug account creation at each lease
authorfsaintma <frederic.saint-marcel@inria.fr>
Thu, 8 Jan 2015 16:13:29 +0000 (17:13 +0100)
committerfsaintma <frederic.saint-marcel@inria.fr>
Thu, 8 Jan 2015 16:13:29 +0000 (17:13 +0100)
sfa/importer/iotlabimporter.py
sfa/iotlab/LDAPapi.py
sfa/iotlab/iotlabslices.py

index 3528db4..25427d1 100644 (file)
@@ -552,7 +552,7 @@ class IotlabImporter:
         # import site and node records in site into the SFA db.
         self.import_sites_and_nodes(testbed_shell)
         #import users and slice into the SFA DB.
-        self.import_persons_and_slices(testbed_shell)
+        #self.import_persons_and_slices(testbed_shell)
 
          ### remove stale records
         # special records must be preserved
index 217e200..4fea094 100644 (file)
@@ -589,51 +589,28 @@ class LDAPapi:
         .. seealso:: make_ldap_filters_from_record
 
         """
-        logger.debug(" \r\n \t LDAP LdapAddUser \r\n\r\n ================\r\n ")
-        user_ldap_attrs = self.make_ldap_attributes_from_record(record)
-        logger.debug("JORDAN LdapAddUser (ctd) user_ldap_attrs=%r" % user_ldap_attrs)
-
-        #Check if user already in LDAP wih email, first name and last name
-        filter_by = self.make_ldap_filters_from_record(user_ldap_attrs)
-        logger.debug("JORDAN LdapAddUser (ctd) filter_by = %r" % filter_by)
-        user_exist = self.LdapSearch(filter_by)
-        logger.debug("JORDAN LdapAddUser (ctd) user_exist = %r" % user_exist)
-        if user_exist:
-            logger.warning(" \r\n \t LDAP LdapAddUser user %s %s \
-                        already exists" % (user_ldap_attrs['sn'],
-                           user_ldap_attrs['mail']))
-            return {'bool': False}
-
-        #Bind to the server
-        result = self.conn.connect()
-        logger.debug("JORDAN LdapAddUser (ctd) result = %r" % result)
-
-        if(result['bool']):
-
-            # A dict to help build the "body" of the object
-            logger.debug(" \r\n \t LDAP LdapAddUser attrs %s "
-                         % user_ldap_attrs)
-
+        filter_by = self.make_ldap_filters_from_record({'email' : record['email']})
+        user = self.LdapSearch(filter_by)
+        if user:
+            logger.debug("LDAPapi.py user ldap exist \t%s" % user)
+            # user = [('uid=saint,ou=People,dc=senslab,dc=info', {'uid': ['saint'], 'givenName': ['Fred'], ...})]
+            return {'bool': True, 'uid': user[0][1]['uid'][0]}
+        else:
+            self.conn.connect()
+            user_ldap_attrs = self.make_ldap_attributes_from_record(record)
+            logger.debug("LDAPapi.py user ldap doesn't exist \t%s" % user_ldap_attrs)
             # The dn of our new entry/object
             dn = 'uid=' + user_ldap_attrs['uid'] + "," + self.baseDN
-
             try:
                 ldif = modlist.addModlist(user_ldap_attrs)
-                logger.debug("LDAPapi.py add attrs %s \r\n  ldif %s"
-                             % (user_ldap_attrs, ldif))
                 self.conn.ldapserv.add_s(dn, ldif)
-
-                logger.info("Adding user %s login %s in LDAP"
-                            % (user_ldap_attrs['cn'], user_ldap_attrs['uid']))
             except ldap.LDAPError, error:
                 logger.log_exc("LDAP Add Error %s" % error)
                 return {'bool': False, 'message': error}
-
             self.conn.close()
             return {'bool': True, 'uid': user_ldap_attrs['uid']}
-        else:
-            return result
-
+        
+        
     def LdapDelete(self, person_dn):
         """Deletes a person in LDAP. Uses the dn of the user.
 
index 7206c92..e24560c 100644 (file)
@@ -359,7 +359,7 @@ class IotlabSlices:
                 logger.debug(" IOTLABSLICES \tverify_slice hrn %s USER %s"
                              % (hrn, user))
 
-                 # add the external slice to the local SFA iotlab DB
+                # add the external slice to the local SFA iotlab DB
                 if sfa_slice:
                     self.driver.AddSlice(sfa_slice, user)
 
@@ -390,222 +390,36 @@ class IotlabSlices:
 
         .. seealso:: AddPerson
         .. note:: Removed unused peer and sfa_peer parameters. SA 18/07/13.
-
-
         """
-        slice_user = slice_record['user']['hrn']
-
         if options is None: options={}
-        logger.debug("IOTLABSLICES \tverify_persons \tslice_hrn  %s  \
-                    \t slice_record %s\r\n users %s \t  "
-                     % (slice_hrn, slice_record, users))
-
-        users_by_email = {}
-        #users_dict : dict whose keys can either be the user's hrn or its id.
-        #Values contains only id and hrn
-        users_dict = {}
-        
-        # XXX LOIC !!! Fix: Only 1 user per slice in iotlab
-        users = [slice_record['user']]
-        #First create dicts by hrn and id for each user in the user record list:
-        for info in users:
-            # if 'slice_record' in info:
-            #     slice_rec = info['slice_record']
-                # if 'user' in slice_rec :
-                #     user = slice_rec['user']
-
-            if 'email' in info:
-                users_by_email[info['email']] = info
-                users_dict[info['email']] = info
-
-        #logger.debug("IOTLABSLICES.PY \t verify_person  \
-        #                users_dict %s \r\n user_by_email %s \r\n  "
-        #             % (users_dict, users_by_email))
-
-        existing_user_ids = []
-        existing_users_by_email = dict()
-        existing_users = []
-        # Check if user is in Iotlab LDAP using its hrn.
-        # Assuming Iotlab is centralised :  one LDAP for all sites,
-        # user's record_id unknown from LDAP
-        # LDAP does not provide users id, therefore we rely on email to find the
-        # user in LDAP
-
-        if users_by_email:
-            #Construct the list of filters (list of dicts) for GetPersons
-            filter_user = [users_by_email[email] for email in users_by_email]
-            #Check user i in LDAP with GetPersons
-            #Needed because what if the user has been deleted in LDAP but
-            #is still in SFA?
-            # GetPersons -> LdapFindUser -> _process_ldap_info_for_one_user
-            # XXX LOIC Fix in _process_ldap_info_for_one_user not to update user with hrn=None
-            existing_users = self.driver.testbed_shell.GetPersons(filter_user)
-            logger.debug(" \r\n IOTLABSLICES.PY \tverify_person  filter_user %s\
-                       existing_users %s  "
-                        % (filter_user, existing_users))
-            #User is in iotlab LDAP
-            if existing_users:
-                for user in existing_users:
-                    user['login'] = user['uid']
-                    # XXX LOIC Fix we already have all informations comming from Allocate
-                    #users_dict[user['email']].update(user)
-                    existing_users_by_email[user['email']] = user
-                logger.debug("User is in iotlab LDAP slice_record[user] = %s" % slice_user)
-
-            # User from another known trusted federated site. Check
-            # if a iotlab account matching the email has already been created.
-            else:
-                req = 'mail='
-                if isinstance(users, list):
-                    req += users[0]['email']
-                else:
-                    req += users['email']
-                ldap_reslt = self.driver.testbed_shell.ldap.LdapSearch(req)
-                logger.debug("LdapSearch slice_record[user] = %s" % slice_user)
-                if ldap_reslt:
-                    logger.debug(" IOTLABSLICES.PY \tverify_person users \
-                                USER already in Iotlab \t ldap_reslt %s \
-                                " % (ldap_reslt))
-                    existing_users.append(ldap_reslt[1])
-                    logger.debug("ldap_reslt slice_record[user] = %s" % slice_user)
-                else:
-                    #User not existing in LDAP
-                    logger.debug("IOTLABSLICES.PY \tverify_person users \
-                                not in ldap ...NEW ACCOUNT NEEDED %s \r\n \t \
-                                ldap_reslt %s " % (users, ldap_reslt))
-
-        requested_user_emails = users_by_email.keys()
-        # requested_user_hrns = \
-        #     [users_by_email[user]['hrn'] for user in users_by_email]
-        # logger.debug("IOTLABSLICES.PY \tverify_person  \
-        #                users_by_email  %s " % (users_by_email))
-
-        # #Check that the user of the slice in the slice record
-        # #matches one of the existing users
-        # try:
-        #     if slice_record['reg-researchers'][0] in requested_user_hrns:
-        #         logger.debug(" IOTLABSLICES  \tverify_person ['PI']\
-        #                         slice_record %s" % (slice_record))
-
-        # except KeyError:
-        #     pass
-
-        # The function returns a list of added persons (to the LDAP ?)
-        added_persons = list()
-
-        # We go though each requested user and make sure it exists both in the
-        # LDAP and in the local DB
-        for user_email in requested_user_emails:
-            user = users_by_email[user_email]
-
-            person = {
+        user = slice_record['user']
+        logger.debug("IOTLABSLICES \tverify_persons \tuser  %s " % user)
+        person = {
                 'peer_person_id': None,
                 'mail'      : user['email'],
                 'email'     : user['email'],
                 'key_ids'   : user.get('key_ids', []),
-                'hrn'       : users_by_email[user['email']]['hrn'],
-            }
-            if 'first_name' in user:
-                person['first_name'] = user['first_name']
-            if 'last_name' in user:
-                person['last_name'] = user['last_name']
-            if 'person_id' in user:
-                person['person_id'] = user['person_id']
-            if user['keys']:
-                # XXX Only one key is kept for IoTLAB
-                person['pkey'] = user['keys'][0]
-
-            # LDAP 
-            if users_by_email not in existing_users_by_email.keys():
-                ret = self.driver.AddPerson(person)
-                if 'uid' in ret:
-                    person['uid'] = ret['uid']
-                    added_persons.append(person)
-                else:
-                    logger.debug(" IOTLABSLICES ret message %s" %(ret))
-            else:
-                person['uid'] = existing_users_by_email[user['email']]['uid']
-
-            # Local DB
-            self.driver.add_person_to_db(person)
-
-            
-        # Set the login in the slice_record XXX
-        slice_record['login'] = existing_users[0]['uid']
-
-        return added_persons
-
-#DEPRECATED|        # users to be added, removed or updated
-#DEPRECATED|        #One user in one iotlab slice : there should be no need
-#DEPRECATED|        #to remove/ add any user from/to a slice.
-#DEPRECATED|        #However a user from SFA which is not registered in Iotlab yet
-#DEPRECATED|        #should be added to the LDAP.
-#DEPRECATED|        added_user_emails = set(requested_user_emails).\
-#DEPRECATED|                                        difference(set(existing_user_emails))
-#DEPRECATED|
-#DEPRECATED|
-#DEPRECATED|        #self.verify_keys(existing_slice_users, updated_users_list, \
-#DEPRECATED|                                                            #peer, append)
-#DEPRECATED|
-#DEPRECATED|        # XXX JORDAN the uid of the user is put in slice_record['login']
-#DEPRECATED|        added_persons = []
-#DEPRECATED|        # add new users
-#DEPRECATED|        #requested_user_email is in existing_user_emails
-#DEPRECATED|        if len(added_user_emails) == 0:
-#DEPRECATED|            slice_record['login'] = existing_users[0]['uid']
-#DEPRECATED|            #slice_record['login'] = users_dict[requested_user_emails[0]]['uid']
-#DEPRECATED|            logger.debug(" IOTLABSLICES  \tverify_person QUICK DIRTY %s"
-#DEPRECATED|                         % (slice_record))
-#DEPRECATED|            # XXX JORDAN uid == 'register'
-#DEPRECATED|        logger.debug("JORDAN USERS BY EMAIL: %r" % users_by_email)
-#DEPRECATED|
-#DEPRECATED|        # XXX JORDAN i have no added_user_emails
-#DEPRECATED|        logger.debug("JORDAN: added_user_emails: %r" % added_user_emails)
-#DEPRECATED|        for added_user_email in added_user_emails:
-#DEPRECATED|            added_user = users_dict[added_user_email]
-#DEPRECATED|            logger.debug(" IOTLABSLICES \r\n \r\n  \t  verify_person \
-#DEPRECATED|                         added_user %s" % (added_user))
-#DEPRECATED|            person = {}
-#DEPRECATED|            person['peer_person_id'] = None
-#DEPRECATED|            k_list = ['first_name', 'last_name', 'person_id']
-#DEPRECATED|            for k in k_list:
-#DEPRECATED|                if k in added_user:
-#DEPRECATED|                    person[k] = added_user[k]
-#DEPRECATED|            # bug user without key
-#DEPRECATED|            if added_user['keys']:
-#DEPRECATED|                person['pkey'] = added_user['keys'][0]
-#DEPRECATED|            person['mail'] = added_user['email']
-#DEPRECATED|            person['email'] = added_user['email']
-#DEPRECATED|            person['key_ids'] = added_user.get('key_ids', [])
-#DEPRECATED|
-#DEPRECATED|            # JORDAN
-#DEPRECATED|            # This is the only call to AddPerson. We need to be sure to provide
-#DEPRECATED|            # the right hrn, by default it used to be done in the function like
-#DEPRECATED|            # this:
-#DEPRECATED|            # person['hrn'] = self.testbed_shell.root_auth + '.' + ret['uid']
-#DEPRECATED|            person['hrn'] = users_by_email[added_user['email']]['hrn']
-#DEPRECATED|
-#DEPRECATED|            # This only deals with the LDAP (now)
-#DEPRECATED|            ret = self.driver.AddPerson(person)
-#DEPRECATED|            # This will check if we have a record in the local DB and add it if necessary
-#DEPRECATED|            self.__add_person_to_db(person)
-#DEPRECATED|
-#DEPRECATED|            if 'uid' in ret:
-#DEPRECATED|                # meaning bool is True and the AddPerson was successful
-#DEPRECATED|                person['uid'] = ret['uid']
-#DEPRECATED|                slice_record['login'] = person['uid']
-#DEPRECATED|            else:
-#DEPRECATED|                # error message in ret
-#DEPRECATED|                logger.debug(" IOTLABSLICES ret message %s" %(ret))
-#DEPRECATED|
-#DEPRECATED|            logger.debug(" IOTLABSLICES \r\n \r\n  \t THE SECOND verify_person\
-#DEPRECATED|                           person %s" % (person))
-#DEPRECATED|            #Update slice_Record with the id now known to LDAP
-#DEPRECATED|
-#DEPRECATED|
-#DEPRECATED|            added_persons.append(person)
-#DEPRECATED|        return added_persons
-
+                'hrn'       : user['hrn'],
+        }
+        if 'first_name' in user:
+            person['first_name'] = user['first_name']
+        if 'last_name' in user:
+            person['last_name'] = user['last_name']
+        if 'person_id' in user:
+            person['person_id'] = user['person_id']
+        if user['keys']:
+            # Only one key is kept for IoTLAB
+            person['pkey'] = user['keys'][0]
+        # SFA DB (if user already exist we do nothing)
+        self.driver.add_person_to_db(person)
+        # Iot-LAB LDAP (if user already exist we do nothing)
+        ret = self.driver.AddPerson(person)
+        # user uid information is only in LDAP
+        # Be carreful : global scope of dict slice_record in driver
+        slice_record['login'] = ret['uid']
+        return person
+
+       
 
     def verify_keys(self, persons, users, peer, options=None):
         """