From 61eda671eca96b6fa32369572d9b49850895d37b Mon Sep 17 00:00:00 2001 From: Daniel J Walsh Date: Thu, 13 Sep 2018 08:36:49 -0400 Subject: 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 | 10 +++++++--- completions/bash/podman | 2 ++ docs/podman-rm.1.md | 5 +++++ 3 files changed, 14 insertions(+), 3 deletions(-) diff --git a/cmd/podman/rm.go b/cmd/podman/rm.go index eb9f1b170..caf7a5f36 100644 --- a/cmd/podman/rm.go +++ b/cmd/podman/rm.go @@ -12,15 +12,19 @@ import ( var ( rmFlags = []cli.Flag{ + 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: "all, a", - Usage: "Remove all containers", + Name: "volumes, v", + Usage: "Remove the volumes associated with the container (Not implemented yet)", }, - LatestFlag, } 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 91c08a78c..d9af43d37 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 27eda5cdd..7474a0d1f 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 -- cgit v1.2.3-54-g00ecf