-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Feature: provide failFast flag, allow a DAG to run all branches of the DAG (either success or failure) #1442
Comments
@xianlubird this behavior is actually the intended behavior. Essentially, the DAG logic has a built-in "fail fast" feature to stop scheduling new steps, as soon as it detects that one of the DAG nodes is failed. Then it waits until all DAG nodes are completed before failing the DAG itself. The reason that you see difference in behavior affected by the retryStrategy of your workflow, is timing related, more than than the retryStrategy. I believe what you are trying to achieve, is to allow a DAG to run all branches of the DAG to completion (either success or failure), regardless of the failed outcomes of branches in the DAG. For this, we need a new feature to disable the "fail fast" behavior, which would be a new feature. We need a new flag for this, because the fail fast behavior of DAGs is desirable behavior for many use cases, and we do not want to break backwards compatibility. |
Hi Argo experts, I wish to add cron schedule to a child job. Would highly appreciate any help on this |
I have a workflow yaml like this.
dag test case
The dependencies are as follows :
When C node is failed, the workflow will stop at B and won't process on. The output like this:
But If I remove B template
retryStrategy label
the yaml like thisnew test case
The
D E
step will go on whenC
is failed. The output like thisThis seems relate to
retryStrategy
on parent node onB node
.Could help me to solve this problem? Thank you @sarabala1979 @jessesuen
The text was updated successfully, but these errors were encountered: