Skip to content

Commit

Permalink
Merge pull request #858 from linsword13/slurm-analyze
Browse files Browse the repository at this point in the history
Skip using workflow manager status when it fails to resolve
  • Loading branch information
linsword13 authored Jan 31, 2025
2 parents 877c214 + 5c76780 commit bbd8e7e
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions lib/ramble/ramble/application.py
Original file line number Diff line number Diff line change
Expand Up @@ -1771,10 +1771,14 @@ def format_context(context_match, context_format):
success = success and criteria_list.passed()

status = experiment_status.SUCCESS if success else experiment_status.FAILED
# When workflow_manager is present, only use app_status when workflow is completed.
# When workflow_manager is present, only use app_status when workflow is completed or
# unresolved.
if self.workflow_manager is not None:
wm_status = self.workflow_manager.get_status(workspace)
if not (wm_status == experiment_status.COMPLETE or wm_status is None):
if not (
wm_status is None
or wm_status in [experiment_status.COMPLETE, experiment_status.UNRESOLVED]
):
status = wm_status
self.set_status(status)

Expand Down

0 comments on commit bbd8e7e

Please sign in to comment.