The retry counter used in TemporaryReservationManager.Generate() was not
decremented for each failed retry, as presumably intended. This patch
decrements the retry counter after each failed attempt.

Signed-off-by: Apollon Oikonomopoulos <[email protected]>
---
 lib/config.py |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/lib/config.py b/lib/config.py
index 7d81b37..5f83b81 100644
--- a/lib/config.py
+++ b/lib/config.py
@@ -118,6 +118,7 @@ class TemporaryReservationManager:
       new_resource = generate_one_fn()
       if new_resource is not None and new_resource not in all_elems:
         break
+      retries -= 1
     else:
       raise errors.ConfigurationError("Not able generate new resource"
                                       " (last tried: %s)" % new_resource)
-- 
1.7.2.5

Reply via email to