summaryrefslogtreecommitdiff
path: root/libpod/container_internal.go
diff options
context:
space:
mode:
authorMatthew Heon <mheon@redhat.com>2019-09-16 09:22:26 -0400
committerMatthew Heon <mheon@redhat.com>2019-09-16 09:22:26 -0400
commitcabe1345f8d8462421853c864c6d8a898713986c (patch)
tree666d2202b24c53334051602c968ffb6416c6f73c /libpod/container_internal.go
parenta1970e1915fa99c1893bccd3a71a11d2bff77602 (diff)
downloadpodman-cabe1345f8d8462421853c864c6d8a898713986c.tar.gz
podman-cabe1345f8d8462421853c864c6d8a898713986c.tar.bz2
podman-cabe1345f8d8462421853c864c6d8a898713986c.zip
Unmounting a container that is already unmounted is OK
We should not be throwing errors because the operation we wanted to perform is already done. Now, it is definitely strange that a container is actually unmounted, but shows as mounted in the DB - if this reoccurs in a way where we can investigate, it's worth tearing into. Fixes #4033 Signed-off-by: Matthew Heon <mheon@redhat.com>
Diffstat (limited to 'libpod/container_internal.go')
-rw-r--r--libpod/container_internal.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/libpod/container_internal.go b/libpod/container_internal.go
index 6bf8439da..8b96b3f62 100644
--- a/libpod/container_internal.go
+++ b/libpod/container_internal.go
@@ -1368,7 +1368,7 @@ func (c *Container) cleanupStorage() error {
// error
// We still want to be able to kick the container out of the
// state
- if errors.Cause(err) == storage.ErrNotAContainer || errors.Cause(err) == storage.ErrContainerUnknown {
+ if errors.Cause(err) == storage.ErrNotAContainer || errors.Cause(err) == storage.ErrContainerUnknown || errors.Cause(err) == storage.ErrLayerNotMounted {
logrus.Errorf("Storage for container %s has been removed", c.ID())
} else {
if cleanupErr != nil {