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

Merge pull request #6059 from jonyhy96/feat-cmd-flag

Browse Source 
fix: make exec-id flag required in exec command
This commit is contained in:
Derek McGowan
2021-09-28 19:15:54 -07:00
committed by GitHub
parent 0f27a423ef e6ddffc2af
commit d132691f10
1 changed files with 3 additions and 3 deletions
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",