From bdb7b93f25823c4a76d03592ab4c2baa7c30a1b9 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 18 Mar 2019 10:52:18 +0100 Subject: pkg/osutil: kill subprocesses more reliably In some cases we start scp, which starts ssh, then kill scp but the ssh subprocess is not killed. As the result cmd.Wait hangs waiting for EOF on the stdout/stderr, which are still kept alive by ssh subprocess. But ssh just hangs forever. Create a process group for each command and kill whole process group. Hopefully this will help. --- pkg/osutil/osutil.go | 2 ++ 1 file changed, 2 insertions(+) (limited to 'pkg/osutil/osutil.go') diff --git a/pkg/osutil/osutil.go b/pkg/osutil/osutil.go index 7061b02d5..32a8054e7 100644 --- a/pkg/osutil/osutil.go +++ b/pkg/osutil/osutil.go @@ -36,6 +36,7 @@ func Run(timeout time.Duration, cmd *exec.Cmd) ([]byte, error) { if cmd.Stderr == nil { cmd.Stderr = output } + setPdeathsig(cmd) if err := cmd.Start(); err != nil { return nil, fmt.Errorf("failed to start %v %+v: %v", cmd.Path, cmd.Args, err) } @@ -46,6 +47,7 @@ func Run(timeout time.Duration, cmd *exec.Cmd) ([]byte, error) { select { case <-timer.C: timedout <- true + killPgroup(cmd) cmd.Process.Kill() case <-done: timedout <- false -- cgit mrf-deployment