diff options
| author | Matthew Heon <mheon@redhat.com> | 2021-05-26 15:16:35 -0400 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2021-05-26 15:33:28 -0400 |
| commit | fad6e1d3ef334076f180ab8a31ae7fa7a309e096 (patch) | |
| tree | cb2f7ebf05340fdacc92d778338d525a67884853 /cni | |
| parent | ac94be37e996fdebf44e5ace83be5219b9488ec4 (diff) | |
| download | podman-fad6e1d3ef334076f180ab8a31ae7fa7a309e096.tar.gz podman-fad6e1d3ef334076f180ab8a31ae7fa7a309e096.tar.bz2 podman-fad6e1d3ef334076f180ab8a31ae7fa7a309e096.zip | |
Ensure that container still exists when removing
After #8906, there is a potential race condition in container
removal of running containers with `--rm`. Running containers
must first be stopped, which was changed to unlock the container
to allow commands like `podman ps` to continue to run while
stopping; however, this also means that the cleanup process can
potentially run before we re-lock, and remove the container from
under us, resulting in error messages from `podman rm`. The end
result is unchanged, the container is still cleanly removed, but
the `podman rm` command will seem to have failed.
Work around this by pinging the database after we stop the
container to make sure it still exists. If it doesn't, our job is
done and we can exit cleanly.
Signed-off-by: Matthew Heon <mheon@redhat.com>
Diffstat (limited to 'cni')
0 files changed, 0 insertions, 0 deletions