fix: retry strategy being ignored by daemoned nodes. Fix #14715 (#14782)

eduardodbr · web-flow · commit 41e1e8eea386 · 2025-08-28T09:06:46.000+07:00
Signed-off-by: Eduardo Rodrigues &lt;eduardodbr@hotmail.com&gt;
diff --git a/workflow/controller/operator.go b/workflow/controller/operator.go
@@ -2203,7 +2203,7 @@ func (woc *wfOperationCtx) executeTemplate(ctx context.Context, nodeName string,
 		childNodeIDs, lastChildNode := getChildNodeIdsAndLastRetriedNode(retryParentNode, woc.wf.Status.Nodes)
 
 		// The retry node might have completed by now.
-		if retryParentNode.Fulfilled() && woc.childrenFulfilled(retryParentNode) { // if retry node is daemoned we want to check those explicitly
+		if retryParentNode.Fulfilled() && (woc.childrenFulfilled(retryParentNode) || (retryParentNode.IsDaemoned() && retryParentNode.FailedOrError())) { // if retry node is daemoned we want to check those explicitly
 			// If retry node has completed, set the output of the last child node to its output.
 			// Runtime parameters (e.g., `status`, `resourceDuration`) in the output will be used to emit metrics.
 			if lastChildNode != nil {
