From: Eric Price <ecprice@mit.edu>
Date: Wed, 30 Jan 2008 04:35:48 +0000 (-0500)
Subject: Modularize cache_acls.py yet more
X-Git-Tag: sipb-xen-www/1~18
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-web.git/commitdiff_plain/8b2c8fa5de64487465829fee603e4c304ed9f9cc?ds=sidebyside

Modularize cache_acls.py yet more

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

diff --git a/cache_acls.py b/cache_acls.py
index c7484d0..34d5e1e 100644
--- a/cache_acls.py
+++ b/cache_acls.py
@@ -30,24 +30,26 @@ def expandName(name):
         name = 'system:'+name
     return getafsgroups.getAfsGroupMembers(name, 'athena.mit.edu')
 
+def refreshMachine(m):
+    people = set()
+    people.update(expandLocker(m.owner))
+    people.update(expandName(m.administrator))
+    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)
+    
 def refreshCache():
     transaction = ctx.current.create_transaction()
 
     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))
-            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()
+            refreshMachine(m)
+        ctx.current.flush()
             
         # Atomically execute our changes
         transaction.commit()