fix(executor/pns): remove sleep before sigkill (#2995)

argoproj · May 11, 2020 · f952df5 · f952df5
1 parent 2a9ee21
commit f952df5
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/workflow/executor/pns/pns.go b/workflow/executor/pns/pns.go
@@ -281,8 +281,7 @@ func (p *PNSExecutor) killContainer(containerID string) error {
 	if err != executil.ErrWaitPIDTimeout {
 		return err
 	}
-	log.Warnf("Timed out (%v) waiting for pid %d to complete after SIGTERM. Issing SIGKILL", waitPIDOpts.Timeout, pid)
-	time.Sleep(30 * time.Minute)
+	log.Warnf("Timed out (%v) waiting for pid %d to complete after SIGTERM. Issuing SIGKILL", waitPIDOpts.Timeout, pid)
 	err = proc.Signal(syscall.SIGKILL)
 	if err != nil {
 		log.Warnf("Failed to SIGKILL pid %d: %v", pid, err)