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

fix(controller): validation failed when dynamic templateRef is used in nested template #14053

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

chengjoey
Copy link
Member

Fixes #14042 #13773

Motivation

if nested template has reference, input tmpl args

Verification

unit test

…n nested template

Signed-off-by: joey <zchengjoey@gmail.com>
@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
…n nested template (argoproj#14053)

Signed-off-by: joey <zchengjoey@gmail.com>
isubasinghe pushed a commit to pipekit/argo-workflows that referenced this pull request Jan 31, 2025
…n nested template (argoproj#14053)

Signed-off-by: joey <zchengjoey@gmail.com>
isubasinghe pushed a commit that referenced this pull request Jan 31, 2025
…n nested template (#14053)

Signed-off-by: joey <zchengjoey@gmail.com>
Joibel pushed a commit that referenced this pull request Feb 7, 2025
…n nested template (#14053)

Signed-off-by: joey <zchengjoey@gmail.com>
(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
…s used in nested template (argoproj#14053)"

This reverts commit 59026f5.
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 <alan@clucas.org>
Joibel added a commit that referenced this pull request Feb 10, 2025
Signed-off-by: Alan Clucas <alan@clucas.org>
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 <zchengjoey@gmail.com>
(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