-
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
[Proposal] Support resubmit/retry on archived workflows #5405
Labels
area/api
Argo Server API
area/cli
The `argo` CLI
area/ui
area/workflow-archive
good first issue
Good for newcomers
type/feature
Feature request
Projects
Comments
terrytangyuan
changed the title
Support resubmit/retry on archived workflows
[Proposal] Support resubmit/retry on archived workflows
Mar 15, 2021
We should also provide this feature in the UI. I'd split this into two issues:
|
I will take this issue |
Wow this got popular. 👍🏻 @dpadhiar assigned to you. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/api
Argo Server API
area/cli
The `argo` CLI
area/ui
area/workflow-archive
good first issue
Good for newcomers
type/feature
Feature request
Summary
What change needs making?
Support resubmit/retry on archived workflows.
Use Cases
When would you use this?
Currently
argo retry/resubmit
only supports workflows resources that currently exist in the cluster. For clusters where workflows are GC-ed immediately after they are completed, it's not possible to retry/resubmit a workflow. There are workaound for this, e.g. saving the workflow resource manually either as a plain YAML or into archived workflow before we GC it, and then do akubectl create
before we useargo retry/resubmit
. It would be better and less hacky if we have a CLI for this.API/Implementation Discussion
We can implement this in one of the following ways:
argo retry/resubmit wf-name --from-archive
argo archive retry/resubmit
argo retry/resubmit --source=location-to-saved-wf.yaml
Tasks
Any preference?
Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.
The text was updated successfully, but these errors were encountered: