From 61eda671eca96b6fa32369572d9b49850895d37b Mon Sep 17 00:00:00 2001 From: Daniel J Walsh Date: Thu, 13 Sep 2018 08:36:49 -0400 Subject: [PATCH] Add `podman rm --volumes` flag While this is not implemented yet, it is needed for working with existing docker scripts. Signed-off-by: Daniel J Walsh Closes: #1460 Approved by: mheon --- cmd/podman/rm.go | 12 ++++++++---- completions/bash/podman | 2 ++ docs/podman-rm.1.md | 5 +++++ 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/cmd/podman/rm.go b/cmd/podman/rm.go index eb9f1b170f..caf7a5f361 100644 --- a/cmd/podman/rm.go +++ b/cmd/podman/rm.go @@ -12,15 +12,19 @@ import ( var ( rmFlags = []cli.Flag{ - cli.BoolFlag{ - Name: "force, f", - Usage: "Force removal of a running container. The default is false", - }, cli.BoolFlag{ Name: "all, a", Usage: "Remove all containers", }, + cli.BoolFlag{ + Name: "force, f", + Usage: "Force removal of a running container. The default is false", + }, LatestFlag, + cli.BoolFlag{ + Name: "volumes, v", + Usage: "Remove the volumes associated with the container (Not implemented yet)", + }, } rmDescription = fmt.Sprintf(` Podman rm will remove one or more containers from the host. diff --git a/completions/bash/podman b/completions/bash/podman index 91c08a78c8..d9af43d370 100644 --- a/completions/bash/podman +++ b/completions/bash/podman @@ -1744,6 +1744,8 @@ _podman_rm() { -f --latest -l + --volumes + -v " local options_with_args=" diff --git a/docs/podman-rm.1.md b/docs/podman-rm.1.md index 27eda5cdd7..7474a0d1ff 100644 --- a/docs/podman-rm.1.md +++ b/docs/podman-rm.1.md @@ -23,6 +23,11 @@ Remove all containers. Can be used in conjunction with -f as well. Instead of providing the container name or ID, use the last created container. If you use methods other than Podman to run containers such as CRI-O, the last started container could be from either of those methods. + +**--volumes, -v** + +Remove the volumes associated with the container. (Not yet implemented) + ## EXAMPLE podman rm mywebserver