From 62b59df053357f040d85c26727734815046e2bc3 Mon Sep 17 00:00:00 2001 From: baude Date: Fri, 13 Apr 2018 14:26:35 -0500 Subject: Allow the use of -i/-a on any container We used to not allow the use of -a/-i on containers that were not started with -i or a tty. Given the improvements in our terminal handling, this should work now. This also fixes a systemic problem with the autotests. Signed-off-by: baude Closes: #617 Approved by: baude --- cmd/podman/start.go | 16 +--------------- 1 file changed, 1 insertion(+), 15 deletions(-) (limited to 'cmd/podman') diff --git a/cmd/podman/start.go b/cmd/podman/start.go index 597ed29ae..00d153904 100644 --- a/cmd/podman/start.go +++ b/cmd/podman/start.go @@ -3,7 +3,6 @@ package main import ( "fmt" "os" - "strconv" "github.com/pkg/errors" "github.com/projectatomic/libpod/libpod" @@ -91,20 +90,7 @@ func startCmd(c *cli.Context) error { continue } - // We can only be interactive if both the config and the command-line say so - if c.Bool("interactive") && !ctr.Config().Stdin { - return errors.Errorf("the container was not created with the interactive option") - } - - tty, err := strconv.ParseBool(ctr.Spec().Annotations["io.kubernetes.cri-o.TTY"]) - if err != nil { - return errors.Wrapf(err, "unable to parse annotations in %s", ctr.ID()) - } - - // Handle start --attach - // We only get a terminal session if both a tty was specified in the spec and - // -a on the command-line was given. - if attach && tty { + if attach { inputStream := os.Stdin if !c.Bool("interactive") { inputStream = nil -- cgit v1.2.3-54-g00ecf