fix: Fixed mutex with withSequence in http template broken. Fixes arg…

…oproj#12018 Signed-off-by: shmruin <meme_hm@naver.com>
shmruin · Nov 10, 2023 · 182f2c6 · 182f2c6
1 parent 873bacb
commit 182f2c6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/pkg/apis/workflow/v1alpha1/workflow_types.go b/pkg/apis/workflow/v1alpha1/workflow_types.go
@@ -2993,6 +2993,8 @@ func (tmpl *Template) GetNodeType() NodeType {
 		return NodeTypeSteps
 	case TemplateTypeSuspend:
 		return NodeTypeSuspend
+	case TemplateTypeHTTP:
+		return NodeTypeHTTP
 	case TemplateTypePlugin:
 		return NodeTypePlugin
 	}

diff --git a/workflow/controller/http_template.go b/workflow/controller/http_template.go
@@ -8,6 +8,8 @@ func (woc *wfOperationCtx) executeHTTPTemplate(nodeName string, templateScope st
 	node, err := woc.wf.GetNodeByName(nodeName)
 	if err != nil {
 		node = woc.initializeExecutableNode(nodeName, wfv1.NodeTypeHTTP, templateScope, tmpl, orgTmpl, opts.boundaryID, wfv1.NodePending, opts.nodeFlag)
+	}
+	if !node.Fulfilled() {
 		woc.taskSet[node.ID] = *tmpl
 	}
 	return node