diff options
author | Jhon Honce <jhonce@redhat.com> | 2020-09-16 14:30:41 -0700 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2020-09-25 11:19:06 -0400 |
commit | 8ead007e5ad7f2ec41f5368f68df2acd6f9e08b8 (patch) | |
tree | 851223c22fd5286ae7eb4f6cbcd50d7df0e7ffdb | |
parent | 5de9b2b5120780c47fcc15e48796c7656563dda7 (diff) | |
download | podman-8ead007e5ad7f2ec41f5368f68df2acd6f9e08b8.tar.gz podman-8ead007e5ad7f2ec41f5368f68df2acd6f9e08b8.tar.bz2 podman-8ead007e5ad7f2ec41f5368f68df2acd6f9e08b8.zip |
Evict containers before removing via V2 API
Fixes #7535
Signed-off-by: Jhon Honce <jhonce@redhat.com>
-rw-r--r-- | pkg/api/handlers/compat/containers.go | 22 |
1 files changed, 20 insertions, 2 deletions
diff --git a/pkg/api/handlers/compat/containers.go b/pkg/api/handlers/compat/containers.go index 63c44eaef..3a904ba87 100644 --- a/pkg/api/handlers/compat/containers.go +++ b/pkg/api/handlers/compat/containers.go @@ -17,6 +17,7 @@ import ( "github.com/docker/go-connections/nat" "github.com/gorilla/schema" "github.com/pkg/errors" + "github.com/sirupsen/logrus" ) func RemoveContainer(w http.ResponseWriter, r *http.Request) { @@ -44,8 +45,25 @@ func RemoveContainer(w http.ResponseWriter, r *http.Request) { runtime := r.Context().Value("runtime").(*libpod.Runtime) name := utils.GetName(r) con, err := runtime.LookupContainer(name) - if err != nil { - utils.ContainerNotFound(w, name, err) + if err != nil && errors.Cause(err) == define.ErrNoSuchCtr { + // Failed to get container. If force is specified, get the container's ID + // and evict it + if !query.Force { + utils.ContainerNotFound(w, name, err) + return + } + + if _, err := runtime.EvictContainer(r.Context(), name, query.Vols); err != nil { + if errors.Cause(err) == define.ErrNoSuchCtr { + logrus.Debugf("Ignoring error (--allow-missing): %q", err) + w.WriteHeader(http.StatusNoContent) + return + } + logrus.Warn(errors.Wrapf(err, "Failed to evict container: %q", name)) + utils.InternalServerError(w, err) + return + } + w.WriteHeader(http.StatusNoContent) return } |