don't calculate status twice

Netflix · Dec 16, 2024 · 1988741 · 1988741
1 parent c89086d
commit 1988741
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/metaflow/plugins/argo/argo_workflows_deployer_objects.py b/metaflow/plugins/argo/argo_workflows_deployer_objects.py
@@ -206,7 +206,8 @@ def is_running(self):
             True if the workflow status is either 'Pending' or 'Running',
             False otherwise.
         """
-        return self.status is not None and self.status in ["Pending", "Running"]
+        workflow_status = self.status
+        return workflow_status is not None and workflow_status in ["Pending", "Running"]
 
     @property
     def status(self) -> Optional[str]: