fix: mark task result completed use nodeId instead of podname. Fixes: #…

…12733 (#12755) Signed-off-by: shuangkun <tsk2013uestc@163.com>
argoproj · Mar 12, 2024 · d95791f · d95791f
1 parent 03f9f75
commit d95791f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/workflow/controller/operator.go b/workflow/controller/operator.go
@@ -1387,7 +1387,7 @@ func (woc *wfOperationCtx) assessNodeStatus(pod *apiv1.Pod, old *wfv1.NodeStatus
 
 	if x, ok := pod.Annotations[common.AnnotationKeyReportOutputsCompleted]; ok {
 		woc.log.Warn("workflow uses legacy/insecure pod patch, see https://argo-workflows.readthedocs.io/en/release-3.5/workflow-rbac/")
-		resultName := pod.GetName()
+		resultName := woc.nodeID(pod)
 		if x == "true" {
 			woc.wf.Status.MarkTaskResultComplete(resultName)
 		} else {