vbatts/cri-o
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

execsync: Set terminal to true when we pass -t to conmon

Browse source 
We may consider setting it to true all the time but this
should match our previous behavior before we started
using process json for exec.

Signed-off-by: Mrunal Patel <mrunalp@gmail.com>
This commit is contained in:
Mrunal Patel 2018-02-27 15:57:03 -08:00
parent 320a102c1c
commit 30af659b92
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
oci/oci.go
View file

@ -441,7 +441,7 @@ func (r *Runtime) ExecSync(c *Container, command []string, timeout int64) (resp
args = append(args, "-l", logPath) args = append(args, "-l", logPath)
args = append(args, "--socket-dir-path", ContainerAttachSocketDir) args = append(args, "--socket-dir-path", ContainerAttachSocketDir)
processFile, err := PrepareProcessExec(c, command, false) processFile, err := PrepareProcessExec(c, command, c.terminal)
if err != nil { if err != nil {
return nil, ExecSyncError{ return nil, ExecSyncError{
ExitCode: -1, ExitCode: -1,