Errors raised during reservation now have a 'reservation' attribute

seantis · Apr 27, 2016 · a36cad3 · a36cad3
1 parent 3b33cce
commit a36cad3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/HISTORY.rst b/HISTORY.rst
@@ -1,6 +1,9 @@
 Changelog
 ---------
 
+- Errors raised during reservation now have a 'reservation' attribute.
+  [href]
+
 0.2.0 (2016-04-26)
 ~~~~~~~~~~~~~~~~~~~
 

diff --git a/libres/db/scheduler.py b/libres/db/scheduler.py
@@ -1051,10 +1051,13 @@ def approve_reservations(self, token):
             assert reservation.session_id is None, """
                 Reservations must be confirmed before they can be approved.
             """
-
-            slots_to_reserve.extend(
-                self._approve_reservation_record(reservation)
-            )
+            try:
+                slots_to_reserve.extend(
+                    self._approve_reservation_record(reservation)
+                )
+            except errors.LibresError as e:
+                e.reservation = reservation
+                raise e
 
         events.on_reservations_approved(self.context, reservations)