Merge pull request #97888 from pacoxu/fix/97565

check containerd as well as docker-containerd
This commit is contained in:
Kubernetes Prow Robot 2021-02-09 23:46:59 -08:00 committed by GitHub
commit d819199065
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -70,10 +70,10 @@ import (
)
const (
dockerProcessName = "dockerd"
dockerProcessName = "dockerd"
// dockerd option --pidfile can specify path to use for daemon PID file, pid file path is default "/var/run/docker.pid"
dockerPidFile = "/var/run/docker.pid"
containerdProcessName = "docker-containerd"
containerdPidFile = "/run/docker/libcontainerd/docker-containerd.pid"
containerdProcessName = "containerd"
maxPidFileLength = 1 << 10 // 1KB
)
@ -884,6 +884,8 @@ func EnsureDockerInContainer(dockerAPIVersion *utilversion.Version, oomScoreAdj
type process struct{ name, file string }
dockerProcs := []process{{dockerProcessName, dockerPidFile}}
if dockerAPIVersion.AtLeast(containerdAPIVersion) {
// By default containerd is started separately, so there is no pid file.
containerdPidFile := ""
dockerProcs = append(dockerProcs, process{containerdProcessName, containerdPidFile})
}
var errs []error