github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #1306 from estesp/error-no-container

Browse Source 
Delete command should require at least one container ID
This commit is contained in:
Michael Crosby 2017-08-08 11:29:56 -04:00 committed by GitHub
commit 1f04eddad1
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
cmd/ctr/container_delete.go
View File

@ -2,6 +2,7 @@ package main
import (
"context"
"errors"
"fmt"
"github.com/containerd/containerd"
@ -11,8 +12,8 @@ import (
var containersDeleteCommand = cli.Command{
Name: "delete",
Usage: "delete an existing container",
ArgsUsage: "CONTAINER",
Usage: "deletes one or more existing containers",
ArgsUsage: "CONTAINER [CONTAINER, ...]",
Aliases: []string{"del", "rm"},
Flags: []cli.Flag{
cli.BoolFlag{
@ -33,6 +34,9 @@ var containersDeleteCommand = cli.Command{
deleteOpts = append(deleteOpts, containerd.WithSnapshotCleanup)
}
if context.NArg() == 0 {
return errors.New("must specify at least one container to delete")
}
for _, arg := range context.Args() {
if err := deleteContainer(ctx, client, arg, deleteOpts...); err != nil {
if exitErr == nil {