X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fmanagers%2Fregistry_manager.py;h=f6f55edab33d14d8bca632e49b96969bfcae3f8b;hb=6872209af2d5ec927820f2b333b8d3b112260957;hp=d82ef57785bd7ef40c00e72faa01fa226fe30be0;hpb=b9ae62cc0463f168771bf3d4ebf348bb920acb4f;p=sfa.git diff --git a/sfa/managers/registry_manager.py b/sfa/managers/registry_manager.py index d82ef577..f6f55eda 100644 --- a/sfa/managers/registry_manager.py +++ b/sfa/managers/registry_manager.py @@ -9,7 +9,6 @@ from sfa.util.faults import RecordNotFound, AccountNotEnabled, PermissionError, from sfa.util.sfatime import utcparse, datetime_to_epoch from sfa.util.prefixTree import prefixTree from sfa.util.xrn import Xrn, get_authority, hrn_to_urn, urn_to_hrn -from sfa.util.plxrn import hrn_to_pl_login_base from sfa.util.version import version_core from sfa.util.sfalogging import logger @@ -18,8 +17,12 @@ from sfa.trust.credential import Credential from sfa.trust.certificate import Certificate, Keypair, convert_public_key from sfa.trust.gid import create_uuid -from sfa.storage.persistentobjs import make_record,RegRecord +from sfa.storage.model import make_record, RegRecord, RegAuthority, RegUser, RegSlice, RegKey, \ + augment_with_sfa_builtins from sfa.storage.alchemy import dbsession +### the types that we need to exclude from sqlobjects before being able to dump +# them on the xmlrpc wire +from sqlalchemy.orm.collections import InstrumentedList class RegistryManager: @@ -37,7 +40,7 @@ class RegistryManager: 'urn':xrn.get_urn(), 'peers':peers}) - def GetCredential(self, api, xrn, type, is_self=False): + def GetCredential(self, api, xrn, type, caller_xrn=None): # convert xrn to hrn if type: hrn = urn_to_hrn(xrn)[0] @@ -53,30 +56,30 @@ class RegistryManager: record=dbsession.query(RegRecord).filter_by(type=type,hrn=hrn).first() if not record: raise RecordNotFound("hrn=%s, type=%s"%(hrn,type)) - - # verify_cancreate_credential requires that the member lists - # (researchers, pis, etc) be filled in - logger.debug("get credential before augment dict, keys=%s"%record.__dict__.keys()) - self.driver.augment_records_with_testbed_info (record.__dict__) - logger.debug("get credential after augment dict, keys=%s"%record.__dict__.keys()) - if not self.driver.is_enabled (record.__dict__): - raise AccountNotEnabled(": PlanetLab account %s is not enabled. Please contact your site PI" %(record.email)) - + # get the callers gid - # if this is a self cred the record's gid is the caller's gid - if is_self: + # if caller_xrn is not specified assume the caller is the record + # object itself. + if not caller_xrn: caller_hrn = hrn caller_gid = record.get_gid_object() else: - caller_gid = api.auth.client_cred.get_gid_caller() - caller_hrn = caller_gid.get_hrn() - + caller_hrn, caller_type = urn_to_hrn(caller_xrn) + if caller_type: + caller_record = dbsession.query(RegRecord).filter_by(hrn=caller_hrn,type=caller_type).first() + else: + caller_record = dbsession.query(RegRecord).filter_by(hrn=caller_hrn).first() + if not caller_record: + raise RecordNotFound("Unable to associated caller (hrn=%s, type=%s) with credential for (hrn: %s, type: %s)"%(caller_hrn, caller_type, hrn, type)) + caller_gid = GID(string=caller_record.gid) + object_hrn = record.get_gid_object().get_hrn() - rights = api.auth.determine_user_rights(caller_hrn, record.__dict__) + # call the builtin authorization/credential generation engine + rights = api.auth.determine_user_rights(caller_hrn, record) # make sure caller has rights to this object if rights.is_empty(): - raise PermissionError(caller_hrn + " has no rights to " + record.hrn) - + raise PermissionError("%s has no rights to %s (%s)" % \ + (caller_hrn, object_hrn, xrn)) object_gid = GID(string=record.gid) new_cred = Credential(subject = object_gid.get_subject()) new_cred.set_gid_caller(caller_gid) @@ -98,13 +101,14 @@ class RegistryManager: return new_cred.save_to_string(save_parents=True) - def Resolve(self, api, xrns, type=None, full=True): + # the default for full, which means 'dig into the testbed as well', should be false + def Resolve(self, api, xrns, type=None, details=False): if not isinstance(xrns, types.ListType): - xrns = [xrns] # try to infer type if not set and we get a single input if not type: type = Xrn(xrns).get_type() + xrns = [xrns] hrns = [urn_to_hrn(xrn)[0] for xrn in xrns] # load all known registry names into a prefix tree and attempt to find @@ -136,7 +140,9 @@ class RegistryManager: credential = api.getCredential() interface = api.registries[registry_hrn] server_proxy = api.server_proxy(interface, credential) - peer_records = server_proxy.Resolve(xrns, credential,type) + # should propagate the details flag but that's not supported in the xmlrpc interface yet + #peer_records = server_proxy.Resolve(xrns, credential,type, details=details) + peer_records = server_proxy.Resolve(xrns, credential) # pass foreign records as-is # previous code used to read # records.extend([SfaRecord(dict=record).as_dict() for record in peer_records]) @@ -149,12 +155,16 @@ class RegistryManager: local_records = dbsession.query(RegRecord).filter(RegRecord.hrn.in_(local_hrns)) if type: local_records = local_records.filter_by(type=type) - local_records=local_records.all() - logger.info("Resolve: local_records=%s (type=%s)"%(local_records,type)) + local_records=local_records.all() + + for local_record in local_records: + augment_with_sfa_builtins (local_record) + + logger.info("Resolve, (details=%s,type=%s) local_records=%s "%(details,type,local_records)) local_dicts = [ record.__dict__ for record in local_records ] - if full: - # in full mode we get as much info as we can, which involves contacting the + if details: + # in details mode we get as much info as we can, which involves contacting the # testbed for getting implementation details about the record self.driver.augment_records_with_testbed_info(local_dicts) # also we fill the 'url' field for known authorities @@ -167,21 +177,21 @@ class RegistryManager: record.url=neighbour_dict[hrn].get_url() return for record in local_records: solve_neighbour_url (record) - # convert local record objects to dicts for xmlrpc # xxx somehow here calling dict(record) issues a weird error # however record.todict() seems to work fine # records.extend( [ dict(record) for record in local_records ] ) - records.extend( [ record.todict() for record in local_records ] ) + records.extend( [ record.todict(exclude_types=[InstrumentedList]) for record in local_records ] ) + if not records: raise RecordNotFound(str(hrns)) return records - def List (self, api, xrn, origin_hrn=None): - hrn, type = urn_to_hrn(xrn) + def List (self, api, xrn, origin_hrn=None, options={}): # load all know registry names into a prefix tree and attempt to find # the longest matching prefix + hrn, type = urn_to_hrn(xrn) registries = api.registries registry_hrns = registries.keys() tree = prefixTree() @@ -198,17 +208,29 @@ class RegistryManager: credential = api.getCredential() interface = api.registries[registry_hrn] server_proxy = api.server_proxy(interface, credential) - record_list = server_proxy.List(xrn, credential) + record_list = server_proxy.List(xrn, credential, options) # same as above, no need to process what comes from through xmlrpc # pass foreign records as-is record_dicts = record_list # if we still have not found the record yet, try the local registry if not record_dicts: + recursive = False + if ('recursive' in options and options['recursive']): + recursive = True + elif hrn.endswith('*'): + hrn = hrn[:-1] + recursive = True + if not api.auth.hierarchy.auth_exists(hrn): raise MissingAuthority(hrn) - records = dbsession.query(RegRecord).filter_by(authority=hrn) - record_dicts=[ record.todict() for record in records ] + if recursive: + records = dbsession.query(RegRecord).filter(RegRecord.hrn.startswith(hrn)) + else: + records = dbsession.query(RegRecord).filter_by(authority=hrn) + # so that sfi list can show more than plain names... + for record in records: augment_with_sfa_builtins (record) + record_dicts=[ record.todict(exclude_types=[InstrumentedList]) for record in records ] return record_dicts @@ -227,21 +249,22 @@ class RegistryManager: #################### # utility for handling relationships among the SFA objects - # given that the SFA db does not handle this sort of relationsships - # it will rely on side-effects in the testbed to keep this persistent # subject_record describes the subject of the relationships # ref_record contains the target values for the various relationships we need to manage - # (to begin with, this is just the slice x person relationship) - def update_relations (self, subject_obj, ref_obj): + # (to begin with, this is just the slice x person (researcher) and authority x person (pi) relationships) + def update_driver_relations (self, subject_obj, ref_obj): type=subject_obj.type - if type=='slice': - self.update_relation(subject_obj, 'researcher', ref_obj.researcher, 'user') + #for (k,v) in subject_obj.__dict__.items(): print k,'=',v + if type=='slice' and hasattr(ref_obj,'researcher'): + self.update_driver_relation(subject_obj, ref_obj.researcher, 'user', 'researcher') + elif type=='authority' and hasattr(ref_obj,'pi'): + self.update_driver_relation(subject_obj,ref_obj.pi, 'user', 'pi') # field_key is the name of one field in the record, typically 'researcher' for a 'slice' record # hrns is the list of hrns that should be linked to the subject from now on # target_type would be e.g. 'user' in the 'slice' x 'researcher' example - def update_relation (self, record_obj, field_key, hrns, target_type): + def update_driver_relation (self, record_obj, hrns, target_type, relation_name): # locate the linked objects in our db subject_type=record_obj.type subject_id=record_obj.pointer @@ -249,7 +272,7 @@ class RegistryManager: link_id_tuples = dbsession.query(RegRecord.pointer).filter_by(type=target_type).filter(RegRecord.hrn.in_(hrns)).all() # sqlalchemy returns named tuples for columns link_ids = [ tuple.pointer for tuple in link_id_tuples ] - self.driver.update_relation (subject_type, target_type, subject_id, link_ids) + self.driver.update_relation (subject_type, target_type, relation_name, subject_id, link_ids) def Register(self, api, record_dict): @@ -286,15 +309,30 @@ class RegistryManager: gid = gid_object.save_to_string(save_parents=True) record.gid = gid - if type in ["authority"]: + if isinstance (record, RegAuthority): # update the tree if not api.auth.hierarchy.auth_exists(hrn): api.auth.hierarchy.create_auth(hrn_to_urn(hrn,'authority')) # get the GID from the newly created authority + auth_info = api.auth.get_auth_info(hrn) gid = auth_info.get_gid_object() record.gid=gid.save_to_string(save_parents=True) + # locate objects for relationships + pi_hrns = getattr(record,'pi',None) + if pi_hrns is not None: record.update_pis (pi_hrns) + + elif isinstance (record, RegSlice): + researcher_hrns = getattr(record,'researcher',None) + if researcher_hrns is not None: record.update_researchers (researcher_hrns) + + elif isinstance (record, RegUser): + # create RegKey objects for incoming keys + if hasattr(record,'keys'): + logger.debug ("creating %d keys for user %s"%(len(record.keys),record.hrn)) + record.reg_keys = [ RegKey (key) for key in record.keys ] + # update testbed-specific data if needed pointer = self.driver.register (record.__dict__, hrn, pub_key) @@ -303,15 +341,14 @@ class RegistryManager: dbsession.commit() # update membership for researchers, pis, owners, operators - self.update_relations (record, record) + self.update_driver_relations (record, record) return record.get_gid_object().save_to_string(save_parents=True) def Update(self, api, record_dict): assert ('type' in record_dict) - new_record=RegRecord(dict=record_dict) - type = new_record.type - hrn = new_record.hrn + new_record=make_record(dict=record_dict) + (type,hrn) = (new_record.type, new_record.hrn) # make sure the record exists record = dbsession.query(RegRecord).filter_by(type=type,hrn=hrn).first() @@ -319,15 +356,11 @@ class RegistryManager: raise RecordNotFound("hrn=%s, type=%s"%(hrn,type)) record.just_updated() - # validate the type - if type not in ['authority', 'slice', 'node', 'user']: - raise UnknownSfaType(type) - # Use the pointer from the existing record, not the one that the user # gave us. This prevents the user from inserting a forged pointer pointer = record.pointer - # is the a change in keys ? + # is there a change in keys ? new_key=None if type=='user': if getattr(new_key,'keys',None): @@ -335,10 +368,6 @@ class RegistryManager: if isinstance (new_key,types.ListType): new_key=new_key[0] - # update the PLC information that was specified with the record - if not self.driver.update (record.__dict__, new_record.__dict__, hrn, new_key): - logger.warning("driver.update failed") - # take new_key into account if new_key: # update the openssl key and gid @@ -350,8 +379,32 @@ class RegistryManager: record.gid = gid dsession.commit() + # xxx should do side effects from new_record to record + # not too sure how to do that + # not too big a deal with planetlab as the driver is authoritative, but... + + # update native relations + if isinstance (record, RegSlice): + researcher_hrns = getattr(new_record,'researcher',None) + if researcher_hrns is not None: record.update_researchers (researcher_hrns) + dbsession.commit() + + elif isinstance (record, RegAuthority): + pi_hrns = getattr(new_record,'pi',None) + if pi_hrns is not None: record.update_pis (pi_hrns) + dbsession.commit() + + # update the PLC information that was specified with the record + # xxx oddly enough, without this useless statement, + # record.__dict__ as received by the driver seems to be off + # anyway the driver should receive an object + # (and then extract __dict__ itself if needed) + print "DO NOT REMOVE ME before driver.update, record=%s"%record + if not self.driver.update (record.__dict__, new_record.__dict__, hrn, new_key): + logger.warning("driver.update failed") + # update membership for researchers, pis, owners, operators - self.update_relations (record, new_record) + self.update_driver_relations (record, new_record) return 1