From: Eric Price <ecprice@mit.edu>
Date: Fri, 25 Jan 2008 02:32:42 +0000 (-0500)
Subject: Only update the differences (and actually work).
X-Git-Tag: sipb-xen-www/1~21
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-web.git/commitdiff_plain/92dd742e75e10d804cf1f0010c3b18e39902cdb5

Only update the differences (and actually work).


svn path=/trunk/web/; revision=260
---

diff --git a/cache_acls.py b/cache_acls.py
index e0094e8..ca0b7c9 100644
--- a/cache_acls.py
+++ b/cache_acls.py
@@ -35,11 +35,6 @@ if __name__ == '__main__':
     
     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:
@@ -47,7 +42,11 @@ if __name__ == '__main__':
             people.update(expandLocker(m.owner))
             people.update(expandName(m.administrator))
             print '%s: %s' % (m.name, ' '.join(people))
-            for p in people:
+            old_people = set(a.user for a in m.acl)
+            for removed in old_people - people:
+                ma = [x for x in m.acl if x.user == removed][0]
+                ctx.current.delete(ma)
+            for p in people - old_people:
                 ma = MachineAccess(machine_id=m.machine_id, user=p)
                 ctx.current.save(ma)
             ctx.current.flush()