From 45c47b37d7386feed4217ef731f7fd88ab441104 Mon Sep 17 00:00:00 2001 From: Brandon Squizzato Date: Mon, 13 Dec 2021 11:20:56 -0500 Subject: [PATCH] Don't check 'active' twice --- bonfire/openshift.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bonfire/openshift.py b/bonfire/openshift.py index 1446f38e..a00fa8a8 100644 --- a/bonfire/openshift.py +++ b/bonfire/openshift.py @@ -798,7 +798,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