Merge pull request #19 from bitnami/gonit-ppid-fix

Gonit should set a different PPID for processes started by it
bitnami · Apr 28, 2020 · 6a59ef4 · 6a59ef4
2 parents 9e020b3 + f205ab6
commit 6a59ef4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/main.go b/main.go
@@ -7,7 +7,7 @@ import (
 	"github.com/bitnami/gonit/cmd"
 )
 
-var version = "0.2.3"
+var version = "0.2.4"
 var buildDate = ""
 var commit = ""
 

diff --git a/monitor/checks.go b/monitor/checks.go
@@ -8,6 +8,7 @@ import (
 	"strconv"
 	"strings"
 	"sync"
+	"syscall"
 	"time"
 
 	"github.com/bitnami/gonit/log"
@@ -334,7 +335,10 @@ func newCommand(cmdStr string, timeout time.Duration, opts Opts) *Command {
 func (c *Command) Exec() {
 	// TODO REPORT error, track std streams
 	c.logger.Debugf("/bin/bash -c %s", c.Cmd)
-	c.logger.Debug(exec.Command("/bin/bash", "-c", c.Cmd).Run())
+
+	cmd := exec.Command("/bin/bash", "-c", c.Cmd)
+	cmd.SysProcAttr = &syscall.SysProcAttr{Setpgid: true}
+	c.logger.Debug(cmd.Run())
 }
 
 func formatColumns(len int, args ...interface{}) string {