Remove redundant container exec start
@@ -546,6 +546,7 @@ func (i *dockerInstance) runScriptExec(ctx gocontext.Context, output io.Writer)
AttachStdin: false,
AttachStdout: true,
AttachStderr: true,
+ Detach: false,
Tty: true,
Cmd: i.provider.execCmd,
User: "travis",
@@ -555,16 +556,6 @@ func (i *dockerInstance) runScriptExec(ctx gocontext.Context, output io.Writer)
return &RunResult{Completed: false}, err
}
- execStartOpts := dockertypes.ExecStartCheck{
- Detach: false,
- Tty: true,
- }
-
- err = i.client.ContainerExecStart(ctx, exec.ID, execStartOpts)
- if err != nil {
- return &RunResult{Completed: false}, err
- }
-
hijackedResponse, err := i.client.ContainerExecAttach(ctx, exec.ID, execConfig)
if err != nil {
return &RunResult{Completed: false}, err
