sqlalchemy api changes

diff --git a/debian/changelog b/debian/changelog
index b4da4e2..ae5359d 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+invirt-remote (0.4.15~glasgall4) precise; urgency=low
+
+  * Update for sqlaqlchmy API changes
+
+ -- Adam Glasgall <glasgall@mit.edu>  Sat, 20 Apr 2013 17:36:38 -0400
+

 invirt-remote (0.4.15~glasgall3) precise; urgency=low
 
   * Update for pyroutefs api changes.
 invirt-remote (0.4.15~glasgall3) precise; urgency=low
 
   * Update for pyroutefs api changes.

diff --git a/server/usr/sbin/invirt-remconffs b/server/usr/sbin/invirt-remconffs
index 979e5d0..1cae001 100755 (executable)
--- a/server/usr/sbin/invirt-remconffs
+++ b/server/usr/sbin/invirt-remconffs
@@ -10,6 +10,8 @@ import sqlalchemy as sa
 from invirt import database
 from invirt.config import structs as config
 
 from invirt import database
 from invirt.config import structs as config
 

+import sqlalchemy.orm.util as util
+

 class RemConfFS(routefs.RouteFS):
     """
     RemConfFS creates a filesytem for configuring remctl, like this:
 class RemConfFS(routefs.RouteFS):
     """
     RemConfFS creates a filesytem for configuring remctl, like this:

@@ -50,10 +52,10 @@ class RemConfFS(routefs.RouteFS):
     def getacl(self, machine, **kw):
         """Build the ACL file for a machine
         """
     def getacl(self, machine, **kw):
         """Build the ACL file for a machine
         """

-        s = sa.sql.select([database.machine_access_table.c.user], # Field to select from
+        s = sa.sql.select([database.machine_access_table.class_mapper(type(self)).mapped_table.c.user], # Field to select from

                           sa.sql.and_( # where clause
                           sa.sql.and_( # where clause

-                database.machine_table.c.machine_id==database.machine_access_table.c.machine_id, # join field
-                database.machine_table.c.name == machine), # filter field
+                database.machine_table.class_mapper(type(self)).mapped_table.c.machine_id==database.machine_access_table.class_mapper(type(self)).mapped_table.c.machine_id, # join field
+                database.machine_table.class_mapper(type(self)).mapped_table.c.name == machine), # filter field

                           from_obj=[database.machine_access_table, database.machine_table]) # from tables
         users = [self.userToPrinc(acl[0]) for acl in
                  database.session.execute(s)]
                           from_obj=[database.machine_access_table, database.machine_table]) # from tables
         users = [self.userToPrinc(acl[0]) for acl in
                  database.session.execute(s)]

@@ -71,13 +73,13 @@ class RemConfFS(routefs.RouteFS):
     
     def getmachines(self, **kw):
         """Get the list of VMs in the database. Does not cache to prevent race conditions."""
     
     def getmachines(self, **kw):
         """Get the list of VMs in the database. Does not cache to prevent race conditions."""

-        return list(row[0] for row in database.session.execute(sa.sql.select([database.Machine.c.name])))
+        return list(row[0] for row in database.session.execute(sa.sql.select([database.Machine.class_mapper(type(self)).mapped_table.c.name])))

 
     def getadmin(self, **kw):
         """
         Get the list of administrators for the global ACL.
         """
 
     def getadmin(self, **kw):
         """
         Get the list of administrators for the global ACL.
         """

-        acl = [self.userToPrinc(row[0]) for row in database.session.execute(sa.sql.select([database.admins_table.c.user]))]
+        acl = [self.userToPrinc(row[0]) for row in database.session.execute(sa.sql.select([database.admins_table.class_mapper(type(self)).mapped_table.c.user]))]

         acl.append('include /etc/remctl/acl/web\n')
         return '\n'.join(acl)
     
         acl.append('include /etc/remctl/acl/web\n')
         return '\n'.join(acl)