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

fix: make exec-id flag required in exec command

Browse Source 
Signed-off-by: jonyhy <yun.hao@daocloud.io>
This commit is contained in:
jonyhy 2021-09-28 18:19:32 +08:00
parent 0f27a423ef
commit e6ddffc2af
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cmd/ctr/commands/tasks/exec.go
View File

@ -31,7 +31,6 @@ import (
"github.com/urfave/cli" "github.com/urfave/cli"
) )
//TODO:(jessvalarezo) exec-id is optional here, update to required arg
var execCommand = cli.Command{ var execCommand = cli.Command{
Name: "exec", Name: "exec",
Usage: "execute additional processes in an existing container", Usage: "execute additional processes in an existing container",
@ -51,8 +50,9 @@ var execCommand = cli.Command{
Usage: "detach from the task after it has started execution", Usage: "detach from the task after it has started execution",
}, },
cli.StringFlag{ cli.StringFlag{
Name: "exec-id", Name: "exec-id",
Usage: "exec specific id for the process", Required: true,
Usage: "exec specific id for the process",
}, },
cli.StringFlag{ cli.StringFlag{
Name: "fifo-dir", Name: "fifo-dir",