Terminate all containers within pod after main container completes (#…

…1423) Resolves #1422
argoproj · Jun 18, 2019 · d2f7162 · d2f7162
1 parent 1607d74
commit d2f7162
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/workflow/executor/executor.go b/workflow/executor/executor.go
@@ -1056,10 +1056,6 @@ func (we *WorkflowExecutor) monitorDeadline(ctx context.Context, annotationsUpda
 
 // KillSidecars kills any sidecars to the main container
 func (we *WorkflowExecutor) KillSidecars() error {
-	if len(we.Template.Sidecars) == 0 {
-		log.Infof("No sidecars")
-		return nil
-	}
 	log.Infof("Killing sidecars")
 	pod, err := we.getPod()
 	if err != nil {