From f952df517bae1f423063d61e7542c4f0c4c667e1 Mon Sep 17 00:00:00 2001 From: Adam Gilat Date: Tue, 12 May 2020 00:23:52 +0300 Subject: [PATCH] fix(executor/pns): remove sleep before sigkill (#2995) --- workflow/executor/pns/pns.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/workflow/executor/pns/pns.go b/workflow/executor/pns/pns.go index 14fdd9693e07..7d20666a0f21 100644 --- 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)