Don't check 'active' twice

RedHatInsights · Dec 13, 2021 · 23227ff · 23227ff
1 parent 070a5ed
commit 23227ff
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/bonfire/openshift.py b/bonfire/openshift.py
@@ -780,7 +780,7 @@ def check_for_existing_reservation(requester):
     all_res = get_json("reservation")
 
     for res in all_res["items"]:
-        res_state = res.get("status", {}).get("state") == "active"
+        res_state = res.get("status", {}).get("state")
         if res["spec"]["requester"] == requester and res_state == "active":
             return True