Fix journalctl leak

This fixes the journalctl leak that occurs when a process that uses cadvisor exits. See issues google#1725 and kubernetes/kubernetes#34965.
mtaufen · Aug 23, 2017 · 3190c1c · 3190c1c
1 parent 48f6b99
commit 3190c1c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/utils/oomparser/oomparser.go b/utils/oomparser/oomparser.go
@@ -22,6 +22,7 @@ import (
 	"path"
 	"regexp"
 	"strconv"
+	"syscall"
 	"time"
 
 	"github.com/google/cadvisor/utils"
@@ -167,6 +168,9 @@ func (self *OomParser) StreamOoms(outStream chan *OomInstance) {
 
 func callJournalctl() (io.ReadCloser, error) {
 	cmd := exec.Command("journalctl", "-k", "-f")
+	cmd.SysProcAttr = &syscall.SysProcAttr{
+		Pdeathsig: syscall.SIGKILL,
+	}
 	readcloser, err := cmd.StdoutPipe()
 	if err != nil {
 		return nil, err