cb08f823e3
For operations on multi containers, we printed error for each failed container, then printed an extra message for container names, it seems redundant. Addresses comments: https://github.com/docker/docker/pull/15078#discussion_r47988449 Signed-off-by: Qiang Huang <h.huangqiang@huawei.com> Upstream-commit: e0dc4f27f66d4311238adf4d7027bb3c6b58ad26 Component: engine
34 lines
931 B
Go
34 lines
931 B
Go
package client
|
|
|
|
import (
|
|
"fmt"
|
|
"strings"
|
|
|
|
Cli "github.com/docker/docker/cli"
|
|
flag "github.com/docker/docker/pkg/mflag"
|
|
)
|
|
|
|
// CmdKill kills one or more running container using SIGKILL or a specified signal.
|
|
//
|
|
// Usage: docker kill [OPTIONS] CONTAINER [CONTAINER...]
|
|
func (cli *DockerCli) CmdKill(args ...string) error {
|
|
cmd := Cli.Subcmd("kill", []string{"CONTAINER [CONTAINER...]"}, Cli.DockerCommands["kill"].Description, true)
|
|
signal := cmd.String([]string{"s", "-signal"}, "KILL", "Signal to send to the container")
|
|
cmd.Require(flag.Min, 1)
|
|
|
|
cmd.ParseFlags(args, true)
|
|
|
|
var errs []string
|
|
for _, name := range cmd.Args() {
|
|
if err := cli.client.ContainerKill(name, *signal); err != nil {
|
|
errs = append(errs, fmt.Sprintf("Failed to kill container (%s): %s", name, err))
|
|
} else {
|
|
fmt.Fprintf(cli.out, "%s\n", name)
|
|
}
|
|
}
|
|
if len(errs) > 0 {
|
|
return fmt.Errorf("%s", strings.Join(errs, "\n"))
|
|
}
|
|
return nil
|
|
}
|