Skip to content

Commit

Permalink
[Pipelines] Add Unknown to retryable statuses (mlrun#7231)
Browse files Browse the repository at this point in the history
Currently we only retry on a stable pipeline status (a terminal state
basically), but an "Unknown" state can possibly also be terminated, and
there is no reason to not allow retrying on it.

This PR creates the `retryable_statuses` list, and a followup will use
this list in
[here](https://github.com/mlrun/mlrun/blob/916f8e006b04216883bae9f8487d223935eb7554/server/py/services/api/crud/pipelines.py#L287).
  • Loading branch information
TomerShor authored Feb 4, 2025
1 parent d77b753 commit 33ecd86
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion pipeline-adapters/mlrun-pipelines-kfp-common/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@

setup(
name="mlrun-pipelines-kfp-common",
version="0.3.7",
version="0.3.8",
description="MLRun Pipelines package for providing KFP common functionality",
author="Yaron Haviv",
author_email="yaronh@iguazio.com",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,3 +111,9 @@ def transient_statuses():
for status in RunStatuses.all()
if status not in RunStatuses.stable_statuses()
]

def retryable_statuses(self):
return self.stable_statuses() + [
RunStatuses.unknown,
RunStatuses.runtime_state_unspecified,
]

0 comments on commit 33ecd86

Please sign in to comment.