summaryrefslogtreecommitdiff
path: root/pkg/domain/infra/abi
diff options
context:
space:
mode:
authorDaniel J Walsh <dwalsh@redhat.com>2021-05-21 08:00:47 -0400
committerDaniel J Walsh <dwalsh@redhat.com>2021-05-21 09:37:53 -0400
commit6ca721cccd98bf9936b33d690b28a52cd4213a18 (patch)
treec1c3b9b573a124826258da2a6befc1a83f6873cb /pkg/domain/infra/abi
parente48aa8c82f7f26260f6911bb3ea99ff54c294675 (diff)
downloadpodman-6ca721cccd98bf9936b33d690b28a52cd4213a18.tar.gz
podman-6ca721cccd98bf9936b33d690b28a52cd4213a18.tar.bz2
podman-6ca721cccd98bf9936b33d690b28a52cd4213a18.zip
Drop container does not exist on removal to debugf
We have race conditions where a container can be removed by two different processes when running podman --remove rm. It can be cleaned up in the API or by the conmon executing podman container cleanup. When we fail to remove a container that does not exists we should not be printing errors or warnings, we should just debug the fact. [NO TESTS NEEDED] Since this is a race condition it is difficult to test. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/domain/infra/abi')
-rw-r--r--pkg/domain/infra/abi/containers.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/domain/infra/abi/containers.go b/pkg/domain/infra/abi/containers.go
index d0a2b1bae..237a43441 100644
--- a/pkg/domain/infra/abi/containers.go
+++ b/pkg/domain/infra/abi/containers.go
@@ -275,7 +275,7 @@ func (ic *ContainerEngine) ContainerRm(ctx context.Context, namesOrIds []string,
case nil:
// remove container names that we successfully deleted
reports = append(reports, &report)
- case define.ErrNoSuchCtr:
+ case define.ErrNoSuchCtr, define.ErrCtrExists:
// There is still a potential this is a libpod container
tmpNames = append(tmpNames, ctr)
default: