Changeset 991 for trunk/packages

Timestamp:

Oct 1, 2008, 7:48:39 PM (16 years ago)

Author:

price

Message:

more sqlalchemy-0.4 changes

Location:

trunk/packages

Files:

• sipb-xen-database/debian/changelog (modified) (1 diff)
• sipb-xen-database/python/database/__init__.py (modified) (1 diff)
• sipb-xen-database/python/database/models.py (modified) (5 diffs)
• sipb-xen-remote-server/debian/changelog (modified) (1 diff)

trunk/packages/sipb-xen-database/debian/changelog

@@ -3 +3 @@
   * add remote and console to pg_hba.conf template
   * fix init script so it works
+  * port more code to sqlalchemy 0.4
 
  -- Greg Price <price@mit.edu>  Wed, 01 Oct 2008 19:30:26 -0400

trunk/packages/sipb-xen-database/python/database/__init__.py

@@ -5 +5 @@
 def connect(uri = config.db.uri):
     """ Connect to a given database URI"""
-    meta.bind = sqlalchemy.create_engine(uri)
+    engine = sqlalchemy.create_engine(uri)
+    meta.bind = engine
+    session.configure(bind=engine)

trunk/packages/sipb-xen-database/python/database/models.py

@@ -1 +1 @@
 from sqlalchemy import *
+from sqlalchemy import orm
 from sqlalchemy.orm import create_session, relation
 
@@ -6 +7 @@
 
 __all__ = ['meta',
-           'ctx',
+           'session',
            'clear_cache',
            'machine_table',
@@ -26 +27 @@
 
 meta = ThreadLocalMetaData()
-ctx = SessionContext(create_session)
+session = orm.scoped_session(orm.sessionmaker())
 
 machine_table = Table('machines', meta,
@@ -106 +107 @@
         return "<Autoinstall %s: %s (%s)>" % (self.autoinstall_id, self.description, self.type.type_id)
 
-assign_mapper(ctx, Machine, machine_table,
+session.mapper(Machine, machine_table,
               properties={'nics': relation(NIC, backref="machine", lazy=False),
                           'disks': relation(Disk, backref="machine", lazy=False),
                           'type': relation(Type, lazy=False),
                           'acl': relation(MachineAccess, backref="machine", lazy=False, passive_deletes=True, cascade="all, delete-orphan")});
-assign_mapper(ctx, MachineAccess, machine_access_table)
-assign_mapper(ctx, NIC, nic_table)
-assign_mapper(ctx, Disk, disk_table)
-assign_mapper(ctx, Type, types_table)
-assign_mapper(ctx, CDROM, cdroms_table)
-assign_mapper(ctx, Autoinstall, autoinstalls_table)
+session.mapper(MachineAccess, machine_access_table)
+session.mapper(NIC, nic_table)
+session.mapper(Disk, disk_table)
+session.mapper(Type, types_table)
+session.mapper(CDROM, cdroms_table)
+session.mapper(Autoinstall, autoinstalls_table)
 
 def clear_cache():
@@ -124 +125 @@
     it.  Why is this so obscure?"""
 
-    ctx.registry.clear()
+    # XXX maybe we still need to do this, but it's not doc'd how.
+    #ctx.registry.clear()

trunk/packages/sipb-xen-remote-server/debian/changelog

@@ -1 +1 @@
 sipb-xen-remote-server (0.9) unstable; urgency=low
 
+  [Evan Broder]
   * Make the acl/ subdir of RemConfFS listable
+  
+  [Greg Price]
   * Update RemConfFS for new SQLAlchemy code