From 35a10c9ba541807b96b01cc66f79c72606d9ed99 Mon Sep 17 00:00:00 2001 From: baude Date: Mon, 2 Apr 2018 09:20:13 -0500 Subject: Only allocate tty when -t In our ezrly development, we always allocated a tty when not -d. Now we should only allocated when the user asks for it. Resolves: #573 Signed-off-by: baude Closes: #574 Approved by: rhatdan --- libpod/container_attach.go | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) (limited to 'libpod') diff --git a/libpod/container_attach.go b/libpod/container_attach.go index 3b8f29edb..40d359ddd 100644 --- a/libpod/container_attach.go +++ b/libpod/container_attach.go @@ -83,18 +83,6 @@ func (c *Container) attachContainerSocket(resize <-chan remotecommand.TerminalSi outputStream := os.Stdout errorStream := os.Stderr defer inputStream.Close() - - // TODO Renable this when tty/terminal discussion is had. - /* - tty, err := strconv.ParseBool(c.runningSpec.Annotations["io.kubernetes.cri-o.TTY"]) - if err != nil { - return errors.Wrapf(err, "unable to parse annotations in %s", c.ID) - } - if !tty { - return errors.Errorf("no tty available for %s", c.ID()) - } - */ - if terminal.IsTerminal(int(inputStream.Fd())) { oldTermState, err := term.SaveState(inputStream.Fd()) if err != nil { @@ -104,8 +92,8 @@ func (c *Container) attachContainerSocket(resize <-chan remotecommand.TerminalSi defer term.RestoreTerminal(inputStream.Fd(), oldTermState) } - // Put both input and output into raw - if !noStdIn { + // Put both input and output into raw when we have a terminal + if !noStdIn && c.config.Spec.Process.Terminal { term.SetRawTerminal(inputStream.Fd()) } -- cgit v1.2.3-54-g00ecf