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

Overriding parameter to retry workflow does not work when using WorkflowTempalate #11631

Closed
3 tasks done
toyamagu-2021 opened this issue Aug 20, 2023 · 0 comments · Fixed by #11634
Closed
3 tasks done
Labels

Comments

@toyamagu-2021
Copy link
Member

toyamagu-2021 commented Aug 20, 2023

Pre-requisites

  • I have double-checked my configuration
  • I can confirm the issues exists when I tested with :latest
  • I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?

Description

  • Changes in workflow parameters on retrying are not reflected when submitted from WorkflowTemplate.
    • image
  • spec.arguments.parameters is changed as expected but not for status.storedWorkflowTemplateSpec
  • If we change status.storedWorkflowTemplateSpec manually, it works fine.

Version

latest

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

spec:
  templates:
    - name: argosay
      inputs:
        parameters:
          - name: code
            value: '{{workflow.parameters.code}}'
      outputs: {}
      metadata: {}
      container:
        name: main
        image: argoproj/argosay:v2
        command:
          - sh
          - '-c'
        args:
          - exit '{{inputs.parameters.code}}'
        resources: {}
  entrypoint: argosay
  arguments:
    parameters:
      - name: code

Logs from the workflow controller

kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container

kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded
terrytangyuan pushed a commit that referenced this issue Sep 5, 2023
…1634)

Signed-off-by: toyamagu-2021 toyamagu2021@gmail.com
dpadhiar pushed a commit to dpadhiar/argo-workflows that referenced this issue May 9, 2024
) (argoproj#11634)

Signed-off-by: toyamagu-2021 toyamagu2021@gmail.com
Signed-off-by: Dillen Padhiar <dillen_padhiar@intuit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant