Skip to content

Conversation

@chengjoey
Copy link
Member

Fixes #14042 #13773

Motivation

if nested template has reference, input tmpl args

Verification

unit test

@chengjoey chengjoey force-pushed the fix/dynamic-tmplref branch from 64c1f7b to 972ed7e Compare January 6, 2025 10:58
Copy link
Member

@shuangkun shuangkun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tczhao tczhao enabled auto-merge (squash) January 6, 2025 22:13
Copy link
Member

@tczhao tczhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix

@tczhao tczhao merged commit 82537a7 into argoproj:main Jan 6, 2025
31 checks passed
isubasinghe pushed a commit to pipekit/argo-workflows that referenced this pull request Jan 30, 2025
isubasinghe pushed a commit to pipekit/argo-workflows that referenced this pull request Jan 31, 2025
isubasinghe pushed a commit that referenced this pull request Jan 31, 2025
Joibel pushed a commit that referenced this pull request Feb 7, 2025
…n nested template (#14053)

Signed-off-by: joey <[email protected]>
(cherry picked from commit 82537a7)
@Joibel
Copy link
Member

Joibel commented Feb 10, 2025

This PR causes #14165.

I'm reverting it on main and release-3.6. Could any future fix for this issue include regression tests from #14165 please.

Joibel added a commit to pipekit/argo-workflows that referenced this pull request Feb 10, 2025
Joibel added a commit that referenced this pull request Feb 10, 2025
…s used in nested template (#14053)"

This reverts commit 82537a7.

Signed-off-by: Alan Clucas <[email protected]>
Joibel added a commit that referenced this pull request Feb 10, 2025
tczhao added a commit to tczhao/argo that referenced this pull request Feb 19, 2025
criscola pushed a commit to helio/argo-workflows that referenced this pull request Mar 13, 2025
…n nested template (argoproj#14053)

Signed-off-by: joey <[email protected]>
(cherry picked from commit 82537a7)
criscola pushed a commit to helio/argo-workflows that referenced this pull request Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dynamic template naming on workflowTemplates fails unexpectedly

4 participants