From e6ddffc2af90ce4fad3db6510911bce849b8c578 Mon Sep 17 00:00:00 2001 From: jonyhy Date: Tue, 28 Sep 2021 18:19:32 +0800 Subject: [PATCH] fix: make exec-id flag required in exec command Signed-off-by: jonyhy --- cmd/ctr/commands/tasks/exec.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/ctr/commands/tasks/exec.go b/cmd/ctr/commands/tasks/exec.go index f31a13f00..dd2952536 100644 --- a/cmd/ctr/commands/tasks/exec.go +++ b/cmd/ctr/commands/tasks/exec.go @@ -31,7 +31,6 @@ import ( "github.com/urfave/cli" ) -//TODO:(jessvalarezo) exec-id is optional here, update to required arg var execCommand = cli.Command{ Name: "exec", 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", }, cli.StringFlag{ - Name: "exec-id", - Usage: "exec specific id for the process", + Name: "exec-id", + Required: true, + Usage: "exec specific id for the process", }, cli.StringFlag{ Name: "fifo-dir",