diff options
| author | Matthew Heon <mheon@redhat.com> | 2019-09-16 09:22:26 -0400 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2019-09-16 09:22:26 -0400 |
| commit | cabe1345f8d8462421853c864c6d8a898713986c (patch) | |
| tree | 666d2202b24c53334051602c968ffb6416c6f73c | |
| parent | a1970e1915fa99c1893bccd3a71a11d2bff77602 (diff) | |
| download | podman-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>
| -rw-r--r-- | libpod/container_internal.go | 2 |
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 { |