diff options
author | baude <bbaude@redhat.com> | 2019-02-20 13:46:41 -0600 |
---|---|---|
committer | baude <bbaude@redhat.com> | 2019-02-22 14:13:08 -0600 |
commit | 328250e7bef56b0b69dbea124c1170abd289900f (patch) | |
tree | 03a9d601d3f3ce0ac0d04fc3b2841b970031ed48 /cmd/podman/cleanup.go | |
parent | 8039ccf4d560acbc308066cf8996bc4e6167a4a9 (diff) | |
download | podman-328250e7bef56b0b69dbea124c1170abd289900f.tar.gz podman-328250e7bef56b0b69dbea124c1170abd289900f.tar.bz2 podman-328250e7bef56b0b69dbea124c1170abd289900f.zip |
Improve command line validation
Use the checkallandlatest function to validate flag usage as part
of the cobra command args validation.
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'cmd/podman/cleanup.go')
-rw-r--r-- | cmd/podman/cleanup.go | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/cmd/podman/cleanup.go b/cmd/podman/cleanup.go index 89a4ba050..58ac80624 100644 --- a/cmd/podman/cleanup.go +++ b/cmd/podman/cleanup.go @@ -26,6 +26,9 @@ var ( cleanupCommand.GlobalFlags = MainGlobalOpts return cleanupCmd(&cleanupCommand) }, + Args: func(cmd *cobra.Command, args []string) error { + return checkAllAndLatest(cmd, args, false) + }, Example: `podman container cleanup --latest podman container cleanup ctrID1 ctrID2 ctrID3 podman container cleanup --all`, @@ -49,10 +52,6 @@ func cleanupCmd(c *cliconfig.CleanupValues) error { } defer runtime.Shutdown(false) - if err := checkAllAndLatest(&c.PodmanCommand); err != nil { - return err - } - cleanupContainers, lastError := getAllOrLatestContainers(&c.PodmanCommand, runtime, -1, "all") ctx := getContext() |