Check exec log flags compatibility

mxpv · mxpv · commit 46af8ccd2d44 · 2019-07-02T09:53:11.000-07:00
Signed-off-by: Maksym Pavlenko &lt;makpav@amazon.com&gt;
diff --git a/cmd/ctr/commands/tasks/exec.go b/cmd/ctr/commands/tasks/exec.go
@@ -98,6 +98,15 @@ var execCommand = cli.Command{
 			if err != nil {
 				return err
 			}
+
+			if dir := context.String("fifo-dir"); dir != "" {
+				return errors.New("can't use log-uri with fifo-dir")
+			}
+
+			if tty {
+				return errors.New("can't use log-uri with tty")
+			}
+
 			ioCreator = cio.LogURI(uri)
 		} else {
 			cioOpts := []cio.Opt{cio.WithStdio, cio.WithFIFODir(context.String("fifo-dir"))}
