From 0c1e17c51daccf7632bb62a918a02b30909582ce Mon Sep 17 00:00:00 2001
From: Semjon Kopp <semjon.kopp@sap.com>
Date: Thu, 13 Jun 2019 14:08:14 +0200
Subject: [PATCH] Terminate all containers within pod after main container
 completes

Resolves #1422
---
 workflow/executor/executor.go | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/workflow/executor/executor.go b/workflow/executor/executor.go
index 63d7d9681e80..698b36dc16eb 100644
--- 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 {