advise users to disable the framebuffer

[invirt/packages/invirt-web.git] / validation.py

diff --git a/validation.py b/validation.py
index 07b3f3d..fd2b979 100644 (file)
--- a/validation.py
+++ b/validation.py
@@ -43,7 +43,7 @@ def maxMemory(user, machine=None, on=True):
     if not on:
         return MAX_MEMORY_SINGLE
     machines = getMachinesByOwner(user, machine)
     if not on:
         return MAX_MEMORY_SINGLE
     machines = getMachinesByOwner(user, machine)

-    active_machines = [x for x in machines if g.uptimes[x]]
+    active_machines = [x for x in machines if g.uptimes.get(x)]

     mem_usage = sum([x.memory for x in active_machines if x != machine])
     return min(MAX_MEMORY_SINGLE, MAX_MEMORY_TOTAL-mem_usage)
 
     mem_usage = sum([x.memory for x in active_machines if x != machine])
     return min(MAX_MEMORY_SINGLE, MAX_MEMORY_TOTAL-mem_usage)
 

@@ -55,7 +55,7 @@ def maxDisk(user, machine=None):
 
 def cantAddVm(user):
     machines = getMachinesByOwner(user)
 
 def cantAddVm(user):
     machines = getMachinesByOwner(user)

-    active_machines = [x for x in machines if g.uptimes[x]]
+    active_machines = [x for x in machines if g.uptimes.get(x)]

     if len(machines) >= MAX_VMS_TOTAL:
         return 'You have too many VMs to create a new one.'
     if len(active_machines) >= MAX_VMS_ACTIVE:
     if len(machines) >= MAX_VMS_TOTAL:
         return 'You have too many VMs to create a new one.'
     if len(active_machines) >= MAX_VMS_ACTIVE:

@@ -71,8 +71,6 @@ def validAddVm(user):
 
 def haveAccess(user, machine):
     """Return whether a user has administrative access to a machine"""
 
 def haveAccess(user, machine):
     """Return whether a user has administrative access to a machine"""

-    if user == 'moo':
-        return True

     if user in (machine.administrator, machine.owner):
         return True
     if getafsgroups.checkAfsGroup(user, machine.administrator, 
     if user in (machine.administrator, machine.owner):
         return True
     if getafsgroups.checkAfsGroup(user, machine.administrator, 

@@ -84,8 +82,6 @@ def haveAccess(user, machine):
 
 def owns(user, machine):
     """Return whether a user owns a machine"""
 
 def owns(user, machine):
     """Return whether a user owns a machine"""

-    if user == 'moo':
-        return True

     return not getafsgroups.notLockerOwner(user, machine.owner)
 
 def validMachineName(name):
     return not getafsgroups.notLockerOwner(user, machine.owner)
 
 def validMachineName(name):