fix: patch report outputs completed if task result not exists. (argop…

…roj#12748) Signed-off-by: shuangkun <[email protected]>
akuity · Mar 6, 2024 · 0a75598 · 0a75598
1 parent d27c9d0
commit 0a75598
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/workflow/executor/executor.go b/workflow/executor/executor.go
@@ -798,7 +798,7 @@ func (we *WorkflowExecutor) FinalizeOutput(ctx context.Context) {
 		err := we.patchTaskResultLabels(ctx, map[string]string{
 			common.LabelKeyReportOutputsCompleted: "true",
 		})
-		if apierr.IsForbidden(err) {
+		if apierr.IsForbidden(err) || apierr.IsNotFound(err) {
 			log.WithError(err).Warn("failed to patch task result, falling back to legacy/insecure pod patch, see https://argo-workflows.readthedocs.io/en/latest/workflow-rbac/")
 			// Only added as a backup in case LabelKeyReportOutputsCompleted could not be set
 			err = we.AddAnnotation(ctx, common.AnnotationKeyReportOutputsCompleted, "true")