aboutsummaryrefslogtreecommitdiffstats
path: root/src/workflow
diff options
context:
space:
mode:
authorParker Berberian <pberberian@iol.unh.edu>2019-04-16 16:53:08 +0000
committerGerrit Code Review <gerrit@opnfv.org>2019-04-16 16:53:08 +0000
commit7dc024b48f8d9757a5c43cb5048df2e021f1ad06 (patch)
tree843bc56c42572afa67bb0b38fbd9ecc8d9fc514c /src/workflow
parentd8e9f0e33648426a95ae50bf27fa089036f6a1fd (diff)
parentb3a44b5d833b0d02735bc2c42d5981c0368da7a6 (diff)
Merge "Limit total number of active bookings per user"
Diffstat (limited to 'src/workflow')
-rw-r--r--src/workflow/models.py6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/workflow/models.py b/src/workflow/models.py
index 3784fe1..4ebb042 100644
--- a/src/workflow/models.py
+++ b/src/workflow/models.py
@@ -143,10 +143,12 @@ class BookingAuthManager():
currently checks if the booking uses multiple servers. if it does, then the owner must be a PTL,
which is checked using the provided info file
"""
- if len(booking.resource.template.getHosts()) < 2:
- return True # if they only have one server, we dont care
if booking.owner.userprofile.booking_privledge:
return True # admin override for this user
+ if Booking.objects.filter(owner=booking.owner, end__gt=timezone.now()).count() >= 3:
+ return False
+ if len(booking.resource.template.getHosts()) < 2:
+ return True # if they only have one server, we dont care
if repo.BOOKING_INFO_FILE not in repo.el:
return False # INFO file not provided
ptl_info = self.parse_url(repo.el.get(repo.BOOKING_INFO_FILE))