diff options
| author | Giuseppe Scrivano <gscrivan@redhat.com> | 2021-12-01 13:41:02 +0100 |
|---|---|---|
| committer | Giuseppe Scrivano <gscrivan@redhat.com> | 2021-12-01 13:42:59 +0100 |
| commit | 0afaf78378f8da3a2fb58617a22baed1268b3e7b (patch) | |
| tree | 4cae2ee290de1a7035bc7bfab6ae1609f816906e /libpod | |
| parent | 295a6f7dd086731448a1168a349f62d3035258ca (diff) | |
| download | podman-0afaf78378f8da3a2fb58617a22baed1268b3e7b.tar.gz podman-0afaf78378f8da3a2fb58617a22baed1268b3e7b.tar.bz2 podman-0afaf78378f8da3a2fb58617a22baed1268b3e7b.zip | |
container, cgroup: detect pid termination
If the /proc/$PID/cgroup file doesn't exist, then it is likely the
container was terminated in the meanwhile so report ErrCtrStopped that
is already handled instead of ENOENT.
commit a66f40b4df039e94572fa38c070207a435cfa466 introduced the regression.
Closes: https://github.com/containers/podman/issues/12457
[NO NEW TESTS NEEDED] it solves a race in the CI that is difficult to reproduce.
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Diffstat (limited to 'libpod')
| -rw-r--r-- | libpod/container.go | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libpod/container.go b/libpod/container.go index 482af43f3..2b74a1943 100644 --- a/libpod/container.go +++ b/libpod/container.go @@ -939,6 +939,11 @@ func (c *Container) cGroupPath() (string, error) { procPath := fmt.Sprintf("/proc/%d/cgroup", c.state.PID) lines, err := ioutil.ReadFile(procPath) if err != nil { + // If the file doesn't exist, it means the container could have been terminated + // so report it. + if os.IsNotExist(err) { + return "", errors.Wrapf(define.ErrCtrStopped, "cannot get cgroup path unless container %s is running", c.ID()) + } return "", err } |