Skip to content

Commit 8eb8a1d

Browse files
committed
Merge pull request moby#21709 from crosbymichael/clear-notify-socket
Clear nofity socket from containerd env
2 parents 4a7bd7e + 3654017 commit 8eb8a1d

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

libcontainerd/remote_linux.go

+9
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import (
1010
"os/exec"
1111
"path/filepath"
1212
"strconv"
13+
"strings"
1314
"sync"
1415
"syscall"
1516
"time"
@@ -351,11 +352,19 @@ func (r *remote) runContainerdDaemon() error {
351352
}
352353
logrus.Debugf("runContainerdDaemon: runtimeArgs: %s", args)
353354
}
355+
354356
cmd := exec.Command(containerdBinary, args...)
355357
// redirect containerd logs to docker logs
356358
cmd.Stdout = os.Stdout
357359
cmd.Stderr = os.Stderr
358360
cmd.SysProcAttr = &syscall.SysProcAttr{Setsid: true}
361+
cmd.Env = nil
362+
// clear the NOTIFY_SOCKET from the env when starting containerd
363+
for _, e := range os.Environ() {
364+
if !strings.HasPrefix(e, "NOTIFY_SOCKET") {
365+
cmd.Env = append(cmd.Env, e)
366+
}
367+
}
359368
if err := cmd.Start(); err != nil {
360369
return err
361370
}

0 commit comments

Comments
 (0)