diff options
author | Valentin Rothberg <rothberg@redhat.com> | 2021-12-15 13:06:01 +0100 |
---|---|---|
committer | Valentin Rothberg <rothberg@redhat.com> | 2021-12-16 13:04:58 +0100 |
commit | 3b6510da68c4c72862aaae36b8cc77f4e165a434 (patch) | |
tree | 51b2dd61e753b616c57544518ed9ab6b033addc2 | |
parent | d984fec351c06a95f8a51c7e6ced819c6b17245f (diff) | |
download | podman-3b6510da68c4c72862aaae36b8cc77f4e165a434.tar.gz podman-3b6510da68c4c72862aaae36b8cc77f4e165a434.tar.bz2 podman-3b6510da68c4c72862aaae36b8cc77f4e165a434.zip |
kill: fix output
Make sure that `kill -a` is printing the IDs of killed containers.
Previously, it was only printing empty lines.
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
-rw-r--r-- | cmd/podman/containers/kill.go | 9 | ||||
-rw-r--r-- | test/system/130-kill.bats | 14 |
2 files changed, 20 insertions, 3 deletions
diff --git a/cmd/podman/containers/kill.go b/cmd/podman/containers/kill.go index 449484449..fe4083df8 100644 --- a/cmd/podman/containers/kill.go +++ b/cmd/podman/containers/kill.go @@ -108,10 +108,13 @@ func kill(_ *cobra.Command, args []string) error { return err } for _, r := range responses { - if r.Err == nil { - fmt.Println(r.RawInput) - } else { + switch { + case r.Err != nil: errs = append(errs, r.Err) + case r.RawInput != "": + fmt.Println(r.RawInput) + default: + fmt.Println(r.Id) } } return errs.PrintErrors() diff --git a/test/system/130-kill.bats b/test/system/130-kill.bats index 1ff3a7b61..a9456e03c 100644 --- a/test/system/130-kill.bats +++ b/test/system/130-kill.bats @@ -116,4 +116,18 @@ load helpers is "$output" "Error: valid signals are 1 through 64" "podman create" } +@test "podman kill - print IDs or raw input" { + # kill -a must print the IDs + run_podman run --rm -d $IMAGE top + ctrID="$output" + run_podman kill -a + is "$output" "$ctrID" + + # kill $input must print $input + cname=$(random_string) + run_podman run --rm -d --name $cname $IMAGE top + run_podman kill $cname + is "$output" $cname +} + # vim: filetype=sh |