From: Evan Broder <broder@mit.edu>
Date: Mon, 20 Oct 2008 17:43:09 +0000 (-0400)
Subject: If the ACL for a locker isn't accessible, assume it's empty (closes #82)
X-Git-Tag: sipb-xen-www/3.17~8
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-web.git/commitdiff_plain/55a1b4d83e6a40a12ec3b540692e98c50cef244e

If the ACL for a locker isn't accessible, assume it's empty (closes #82)

svn path=/trunk/packages/sipb-xen-www/; revision=1154
---

diff --git a/code/getafsgroups.py b/code/getafsgroups.py
index 13f8cf7..c775a01 100644
--- a/code/getafsgroups.py
+++ b/code/getafsgroups.py
@@ -48,10 +48,15 @@ def getCell(locker):
     return p.stdout.read().split()[-1][1:-1]
 
 def getLockerAcl(locker):
-    p = subprocess.Popen(["fs", "listacl", getLockerPath(locker)], 
-                         stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-    if p.wait():
-        raise AfsProcessError(p.stderr.read())
+    try:
+        p = subprocess.Popen(["fs", "listacl", getLockerPath(locker)], 
+                             stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+        if p.wait():
+            raise AfsProcessError(p.stderr.read())
+    except AfsProcessError, e:
+        if e.message.startswith("fs: You don't have the required access rights on"):
+            return []
+        raise
     lines = p.stdout.readlines()
     values = []
     for line in lines[1:]: