fix: make exec-id flag required in exec command

jonyhy96 · jonyhy96 · commit e6ddffc2af90 · 2021-09-28T18:19:32.000+08:00
Signed-off-by: jonyhy &lt;yun.hao@daocloud.io&gt;
diff --git 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",
