Changeset 257 for trunk/web

Timestamp:

Jan 24, 2008, 5:20:30 PM (17 years ago)

Author:

ecprice

Message:

Do entire ACL update in a transaction, not for each machine.

File:

• trunk/web/cache_acls.py (modified) (1 diff)

trunk/web/cache_acls.py

@@ -34 +34 @@
     connect('postgres://sipb-xen@sipb-xen-dev/sipb_xen')
     
-    machines = Machine.select()
-    for m in machines:
-        people = set()
-        people.update(expandLocker(m.owner))
-        people.update(expandName(m.administrator))
-        print '%s: %s' % (m.name, ' '.join(people))
-        transaction = ctx.current.create_transaction()
-        for a in m.acl:
-            ctx.current.delete(a)
-        for p in people:
-            ma = MachineAccess(machine_id=m.machine_id, user=p)
-            ctx.current.save(ma)
-        ctx.current.flush()
+    transaction = ctx.current.create_transaction()
+
+    print repr(ctx), repr(ctx.current), repr(transaction)
+    # Remove existing machine access entries
+    machine_access_table.delete()
+    ctx.current.flush()
+
+    try:
+        machines = Machine.select()
+        for m in machines:
+            people = set()
+            people.update(expandLocker(m.owner))
+            people.update(expandName(m.administrator))
+            print '%s: %s' % (m.name, ' '.join(people))
+            for p in people:
+                ma = MachineAccess(machine_id=m.machine_id, user=p)
+                ctx.current.save(ma)
+            ctx.current.flush()
+            
+        # Atomically execute our changes
         transaction.commit()
+    except:
+        # Failed! Rollback all the changes.
+        transaction.rollback()
+        raise