Skip to content

Commit

Permalink
List all info level failure reasons
Browse files Browse the repository at this point in the history
  • Loading branch information
mvdbeek committed Jul 25, 2024
1 parent 433e8ac commit 2b21e05
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 4 deletions.
15 changes: 15 additions & 0 deletions lib/galaxy/schema/invocation.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,21 @@ class FailureReason(str, Enum):
unexpected_failure = "unexpected_failure"


# The reasons below are attached to the invocation and user-actionable.
# Not included are `unexpected_failure` and `expression_evaluation_failed`.
# If expression evaluation fails we're not attaching the templated
# expression to the invocation, as it could contain secrets.
# If the failure reason is not in `FAILURE_REASONS_EXPECTED` we should
# log an exception so admins can debug and/or submit bug reports.
FAILURE_REASONS_EXPECTED = (
FailureReason.dataset_failed,
FailureReason.collection_failed,
FailureReason.job_failed,
FailureReason.output_not_found,
FailureReason.when_not_boolean,
)


class CancelReason(str, Enum):
"""Possible reasons for a cancelled workflow."""

Expand Down
6 changes: 2 additions & 4 deletions lib/galaxy/workflow/run.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
)
from galaxy.schema.invocation import (
CancelReason,
FAILURE_REASONS_EXPECTED,
FailureReason,
InvocationCancellationHistoryDeleted,
InvocationFailureCollectionFailed,
Expand Down Expand Up @@ -252,10 +253,7 @@ def invoke(self) -> Dict[int, Any]:
self.progress.mark_step_outputs_delayed(step, why=de.why)
except Exception as e:
log_function = log.exception
if isinstance(e, modules.FailWorkflowEvaluation) and e.why.reason not in (
"unexpected_failure",
"expression_evaluation_failed",
):
if isinstance(e, modules.FailWorkflowEvaluation) and e.why.reason in FAILURE_REASONS_EXPECTED:
log_function = log.info
log_function(
"Failed to schedule %s for %s, problem occurred on %s.",
Expand Down

0 comments on commit 2b21e05

Please sign in to comment.