X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fstorage%2Fmodel.py;h=7ce834574626a182b1461543c84c69fd06eb0109;hb=c7c7ed1178348445dc4ae70ea0fd905625e95421;hp=c187dcc5e124e45a5a40eb39112b23751cd6bdd6;hpb=d4fdf5099eff793459ad956b6a40ff85003cecc0;p=sfa.git diff --git a/sfa/storage/model.py b/sfa/storage/model.py index c187dcc5..7ce83457 100644 --- a/sfa/storage/model.py +++ b/sfa/storage/model.py @@ -1,6 +1,7 @@ from types import StringTypes from datetime import datetime +from sqlalchemy import or_, and_ from sqlalchemy import Column, Integer, String, DateTime from sqlalchemy import Table, Column, MetaData, join, ForeignKey from sqlalchemy.orm import relationship, backref @@ -17,7 +18,7 @@ from sfa.util.xml import XML from sfa.trust.gid import GID ############################## -Base=declarative_base() +Base = declarative_base() #################### # dicts vs objects @@ -74,7 +75,7 @@ class AlchemyObj(Record): # but we had to define another more internal column (classtype) so we # accomodate variants in types like authority+am and the like -class RegRecord (Base,AlchemyObj): +class RegRecord (Base, AlchemyObj): __tablename__ = 'records' record_id = Column (Integer, primary_key=True) # this is the discriminator that tells which class to use @@ -106,11 +107,19 @@ class RegRecord (Base,AlchemyObj): if dict: self.load_from_dict (dict) def __repr__(self): - result="", " name={}>".format(self.name)) + return result - def update_pis (self, pi_hrns): - # don't ruin the import of that file in a client world - from sfa.storage.alchemy import dbsession + def update_pis (self, pi_hrns, dbsession): # strip that in case we have words pi_hrns = [ x.strip() for x in pi_hrns ] - request = dbsession.query (RegUser).filter(RegUser.hrn.in_(pi_hrns)) - logger.info ("RegAuthority.update_pis: %d incoming pis, %d matches found"%(len(pi_hrns),request.count())) - pis = dbsession.query (RegUser).filter(RegUser.hrn.in_(pi_hrns)).all() + request = dbsession.query(RegUser).filter(RegUser.hrn.in_(pi_hrns)) + logger.info("RegAuthority.update_pis: %d incoming pis, %d matches found"\ + % (len(pi_hrns), request.count())) + pis = dbsession.query(RegUser).filter(RegUser.hrn.in_(pi_hrns)).all() self.reg_pis = pis #################### @@ -209,34 +234,41 @@ class RegSlice (RegRecord): secondary=slice_researcher_table, primaryjoin=RegRecord.record_id==slice_researcher_table.c.slice_id, secondaryjoin=RegRecord.record_id==slice_researcher_table.c.researcher_id, - backref='reg_slices_as_researcher') + backref='reg_slices_as_researcher', + ) def __init__ (self, **kwds): - if 'type' not in kwds: kwds['type']='slice' + if 'type' not in kwds: + kwds['type']='slice' RegRecord.__init__(self, **kwds) def __repr__ (self): - return RegRecord.__repr__(self).replace("Record","Slice") + return RegRecord.__repr__(self).replace("Record", "Slice") - def update_researchers (self, researcher_hrns): - # don't ruin the import of that file in a client world - from sfa.storage.alchemy import dbsession + def update_researchers (self, researcher_hrns, dbsession): # strip that in case we have words researcher_hrns = [ x.strip() for x in researcher_hrns ] request = dbsession.query (RegUser).filter(RegUser.hrn.in_(researcher_hrns)) - logger.info ("RegSlice.update_researchers: %d incoming researchers, %d matches found"%(len(researcher_hrns),request.count())) + logger.info ("RegSlice.update_researchers: %d incoming researchers, %d matches found"\ + % (len(researcher_hrns), request.count())) researchers = dbsession.query (RegUser).filter(RegUser.hrn.in_(researcher_hrns)).all() self.reg_researchers = researchers # when dealing with credentials, we need to retrieve the PIs attached to a slice + # WARNING: with the move to passing dbsessions around, we face a glitch here because this + # helper function is called from the trust/ area that def get_pis (self): - # don't ruin the import of that file in a client world - from sfa.storage.alchemy import dbsession + from sqlalchemy.orm import sessionmaker + Session = sessionmaker() + dbsession = Session.object_session(self) from sfa.util.xrn import get_authority authority_hrn = get_authority(self.hrn) auth_record = dbsession.query(RegAuthority).filter_by(hrn=authority_hrn).first() return auth_record.reg_pis + @validates ('expires') + def validate_expires (self, key, incoming): + return self.validate_datetime (key, incoming) #################### class RegNode (RegRecord): @@ -244,12 +276,13 @@ class RegNode (RegRecord): __mapper_args__ = { 'polymorphic_identity' : 'node' } record_id = Column (Integer, ForeignKey ("records.record_id"), primary_key=True) - def __init__ (self, **kwds): - if 'type' not in kwds: kwds['type']='node' + def __init__(self, **kwds): + if 'type' not in kwds: + kwds['type']='node' RegRecord.__init__(self, **kwds) def __repr__ (self): - return RegRecord.__repr__(self).replace("Record","Node") + return RegRecord.__repr__(self).replace("Record", "Node") #################### class RegUser (RegRecord): @@ -263,20 +296,22 @@ class RegUser (RegRecord): # a 'keys' tag, and assigning a list of strings in a reference column like this crashes reg_keys = relationship \ ('RegKey', backref='reg_user', - cascade="all, delete, delete-orphan") + cascade = "all, delete, delete-orphan", + ) # so we can use RegUser (email=.., hrn=..) and the like def __init__ (self, **kwds): # handle local settings - if 'email' in kwds: self.email=kwds.pop('email') - if 'type' not in kwds: kwds['type']='user' + if 'email' in kwds: + self.email = kwds.pop('email') + if 'type' not in kwds: + kwds['type'] = 'user' RegRecord.__init__(self, **kwds) # append stuff at the end of the record __repr__ def __repr__ (self): - result = RegRecord.__repr__(self).replace("Record","User") - result.replace (">"," email=%s"%self.email) - result += ">" + result = RegRecord.__repr__(self).replace("Record", "User") + result.replace(">", " email={}>".format(self.email)) return result @validates('email') @@ -292,21 +327,103 @@ class RegUser (RegRecord): class RegKey (Base): __tablename__ = 'keys' key_id = Column (Integer, primary_key=True) - record_id = Column (Integer, ForeignKey ("records.record_id")) + record_id = Column (Integer, ForeignKey ("records.record_id")) key = Column (String) pointer = Column (Integer, default = -1) def __init__ (self, key, pointer=None): - self.key=key - if pointer: self.pointer=pointer + self.key = key + if pointer: + self.pointer = pointer def __repr__ (self): - result=":} +# so after that, an 'authority' record will e.g. have a 'reg-pis' field with the hrns of its pi-users +augment_map = {'authority': {'reg-pis' : 'reg_pis',}, + 'slice': {'reg-researchers' : 'reg_researchers',}, + 'user': {'reg-pi-authorities' : 'reg_authorities_as_pi', + 'reg-slices' : 'reg_slices_as_researcher',}, + } + + +def augment_with_sfa_builtins(local_record): + # don't ruin the import of that file in a client world + from sfa.util.xrn import Xrn + # add a 'urn' field + setattr(local_record, 'reg-urn', Xrn(xrn=local_record.hrn, type=local_record.type).urn) + # users have keys and this is needed to synthesize 'users' sent over to CreateSliver + if local_record.type == 'user': + user_keys = [ key.key for key in local_record.reg_keys ] + setattr(local_record, 'reg-keys', user_keys) + # search in map according to record type + type_map = augment_map.get(local_record.type, {}) + # use type-dep. map to do the job + for (field_name, attribute) in type_map.items(): + # get related objects + related_records = getattr(local_record, attribute, []) + hrns = [ r.hrn for r in related_records ] + setattr (local_record, field_name, hrns) + +