diff options
author | chesedo <pieter@chesedo.me> | 2022-06-22 16:52:18 +0200 |
---|---|---|
committer | chesedo <pieter@chesedo.me> | 2022-06-28 15:50:45 +0200 |
commit | cc49146332e279644f1357c6a5e209927bd73471 (patch) | |
tree | 7c646d49ac5947e90ce111e7e4c72ba9ab3c9cfa /test/e2e/pull_test.go | |
parent | 2382955c6a88e506ce6d85477c8c68bd24859bdd (diff) | |
download | podman-cc49146332e279644f1357c6a5e209927bd73471.tar.gz podman-cc49146332e279644f1357c6a5e209927bd73471.tar.bz2 podman-cc49146332e279644f1357c6a5e209927bd73471.zip |
Docker compat returning unknown "initialized" for `status.status`
Some background for this PR is in discussion #14641. In short, ever so often a container inspect will return a `status.status` of `initialized` from the Docker compat socket.
From the discussion I found these lines which tries to fix a "configured" status to "created".
https://github.com/containers/podman/blob/c936d1e61154b6826e9d8df46e9660aba6c86cfe/pkg/api/handlers/compat/containers.go#L291-L294
However, commit 141de8686289 (Revamp Libpod state strings for Docker compat) removed the "configured" return value from the `String()` method called on line 291 above. Thus, making the `if` check redundant as it will never hit. But the same commit also introduces a return for "initialized" which this `if` should probably have been adapted for.
Signed-off-by: Pieter Engelbrecht <pieter@shuttle.rs>
Diffstat (limited to 'test/e2e/pull_test.go')
0 files changed, 0 insertions, 0 deletions