Merge pull request #711 from rushtehrani/fix/send-status

fix: Only inject one sys-send-status task
onepanelio · Nov 9, 2020 · 90f01f8 · 90f01f8
2 parents 850b184 + 875097f
commit 90f01f8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/pkg/workflow_execution.go b/pkg/workflow_execution.go
@@ -1808,13 +1808,13 @@ func injectWorkflowExecutionStatusCaller(wf *wfv1.Workflow, phase wfv1.NodePhase
 				for j, task := range t.DAG.Tasks {
 					if task.Dependencies == nil {
 						wf.Spec.Templates[i].DAG.Tasks[j].Dependencies = []string{containerTemplate.Name}
-						wf.Spec.Templates[i].DAG.Tasks = append(t.DAG.Tasks, wfv1.DAGTask{
-							Name:     containerTemplate.Name,
-							Template: containerTemplate.Name,
-						})
 					}
 				}
 			}
+			wf.Spec.Templates[i].DAG.Tasks = append(t.DAG.Tasks, wfv1.DAGTask{
+				Name:     containerTemplate.Name,
+				Template: containerTemplate.Name,
+			})
 			break
 		}
 	}