-
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
Resubmit/Retry with the same workflow ID #762
Comments
Or probably change the behavior with |
I agree with the use case. Another reason for using the same workflow name, is if artifacts need to be placed together. What about:
Then this flag could be used in non-memoized and memoized mode. e.g.:
|
That sounds good. I'm ok with the proposed change, but it seems to me that this behavior could be the default and would offer a better experience. Is there a specific reason to not have this as the default behavior and have a flag to resubmit under a different workflow name? |
I think you are right.
|
@jessesuen @lippertmarkus : argo retry - will this work even if all the pods(regardless of status) has been deleted but its artifacts had been saved to S3? |
There was a word duplicated in docs. On running codegen I noticed that the mockery command was outdated as well
Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST
Have an
argo retry
orargo resubmit --retry
option to resubmit the workflow without creating a new workflow object. This would allow the retry of workflow from the failed step/task. Whileargo resubmit --memoized
offers retry of a failed workflow, the pods in the resubmitted workflow have reference to the previous workflow's pods. It becomes a nightmare if a user wants to look at the logs of all pods in a workflow. Also, there might be the case that the previous workflows have been deleted and all the pods of the workflows are already deleted.Example of a workflow with multiple references:
Having this feature means that all the information in the workflow remains intact and traceable.
The text was updated successfully, but these errors were encountered: