Skip to content
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

UI: Ability to change parameters when re-submitting workflows #4662

Closed
amoldharamshi opened this issue Dec 7, 2020 · 13 comments · Fixed by #11083
Closed

UI: Ability to change parameters when re-submitting workflows #4662

amoldharamshi opened this issue Dec 7, 2020 · 13 comments · Fixed by #11083
Assignees
Labels
Milestone

Comments

@amoldharamshi
Copy link

Summary

Need ability to change parameters/arguments while resubmitting workflows in Argo UI

Use Case

In case of failures or if we want to run the workflow again with different parameters/arguments, we need ability to do so using Argo UI

@alexec
Copy link
Contributor

alexec commented Dec 7, 2020

Thank you @amoldharamshi I'll speak to my colleagues about your request.

@alexec
Copy link
Contributor

alexec commented Dec 14, 2020

Might be good to fix #3453 at the same time. I'm imagining a new panel that pops up similar to submitting from workflow templates. That would be neat.

@simster7
Copy link
Member

Might be good to fix #3453 at the same time. I'm imagining a new panel that pops up similar to submitting from workflow templates. That would be neat.

Sure thing, I'll take care of this as well

@simster7
Copy link
Member

simster7 commented Jan 3, 2021

I added the ability to set parameters in the workflow submit pane. It's a little trickier to add this in the workflow resubmit pane. I will bring this up with the team tomorrow

@simster7
Copy link
Member

Done

@alexec alexec added good first issue Good for newcomers help wanted labels Jun 8, 2021
@alexec alexec reopened this Jun 8, 2021
@alexec
Copy link
Contributor

alexec commented Jun 8, 2021

Re-opening so we can do this.

@whynowy whynowy removed this from the v3.0 milestone Jun 11, 2021
@alexec alexec removed the epic/unity label Jun 26, 2021
@Guillermogsjc
Copy link

Guillermogsjc commented Jul 30, 2021

Hi @alexec @simster7 ,

it would be really useful to allow to the argo client function resubmit to admit the --parameter flag as does submit.

Something like

argo resubmit -n argo -p UPDATED_PARAMETER=updated_value EXISTING_WORKFLOW_NAME

The use case is pretty clear, you want to retry a workflow changing some parameters.

@alexec
Copy link
Contributor

alexec commented Jul 30, 2021

This issue is about UI. Please raise new issue.

@stale

This comment was marked as resolved.

@stale stale bot added the wontfix label Sep 28, 2021
@alexec alexec added the type/feature Feature request label Sep 28, 2021
@stale stale bot removed the wontfix label Sep 28, 2021
@kennytrytek-wf
Copy link

I'd like to take this one, please. My understanding is that the resubmit UI should have an option in addition to the OK and Cancel buttons to update parameter values before submission. I'm not sure of the best solution for all cases.

For workflow templates and cluster workflow templates, I think it would be good to redirect to the template submission view with the inputs pre-filled on the selected entrypoint, since that would help give a consistent experience submitting workflow templates from the UI.

The difficulty comes for non-template workflows. It seems like we would need a new submit panel that did essentially the same thing as the template submission, but for a workflow that doesn't have a defined template. I don't want to duplicate all the same submit logic in a new place; would it be possible to use the same workflow submission form for both workflow templates and free-form workflows?

@kennytrytek-wf
Copy link

kennytrytek-wf commented Dec 21, 2021

Looking through the code a bit more, this actually seems like it's largely already solved in this case:
https://github.com/argoproj/argo-workflows/blob/master/ui/src/app/workflows/components/workflow-creator.tsx

I'd like to propose the following behavior when clicking the Resubmit button:

  • For a [cluster] workflow template, the SubmitWorkflowPanel side panel is shown, populated with the entrypoint and parameter values that were used in the workflow. Submission will issue a submit request.
  • For a non-templated workflow, the WorkflowEditor side panel is shown, populated with any information available in the executed workflow. Submission will issue a create request.

@alexec alexec added area/ui and removed help wanted labels Feb 2, 2022
toyamagu-2021 added a commit to toyamagu-2021/argo-workflows that referenced this issue May 14, 2023
Signed-off-by: toyamagu2021@gmail.com <toyamagu2021@gmail.com>
toyamagu-2021 added a commit to toyamagu-2021/argo-workflows that referenced this issue May 14, 2023
Signed-off-by: toyamagu2021@gmail.com <toyamagu2021@gmail.com>
@terrytangyuan
Copy link
Member

@toyamagu-2021 Can you comment here so that I can assign this to you?

@toyamagu-2021
Copy link
Member

@terrytangyuan
Thank you for your mention!

toyamagu-2021 added a commit to toyamagu-2021/argo-workflows that referenced this issue May 20, 2023
Signed-off-by: toyamagu2021@gmail.com <toyamagu2021@gmail.com>
terrytangyuan pushed a commit that referenced this issue Jul 16, 2023
Signed-off-by: toyamagu2021@gmail.com <toyamagu2021@gmail.com>
@agilgur5 agilgur5 changed the title Need ability to change parameters/arguments while submitting/re-submitting workflows in Argo UI UI: Ability to change parameters while submitting/re-submitting workflows Jun 24, 2024
@agilgur5 agilgur5 changed the title UI: Ability to change parameters while submitting/re-submitting workflows UI: Ability to change parameters when re-submitting workflows Jun 24, 2024
@agilgur5 agilgur5 added this to the v3.5.0 milestone Jun 24, 2024
@argoproj argoproj locked as resolved and limited conversation to collaborators Jun 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Development

Successfully merging a pull request may close this issue.

10 participants