summaryrefslogtreecommitdiffhomepage
path: root/pkg/test/dockerutil
diff options
context:
space:
mode:
authorZeling Feng <zeling@google.com>2020-11-30 21:21:21 -0800
committergVisor bot <gvisor-bot@google.com>2020-11-30 21:22:55 -0800
commit59a2c785bfba56e2ffb04083c213fa2ad23a36a2 (patch)
treee9c3de0127930f75d693cdf981f7b87fae23d123 /pkg/test/dockerutil
parent79e2364933bbd34dceb0b9025e4e7d145fdf4492 (diff)
Do not start a ContainerExec twice
ContainerExecStart and ContainerExecAttach both call the /exec/id/start API endpoint. PiperOrigin-RevId: 344946627
Diffstat (limited to 'pkg/test/dockerutil')
-rw-r--r--pkg/test/dockerutil/exec.go5
1 files changed, 0 insertions, 5 deletions
diff --git a/pkg/test/dockerutil/exec.go b/pkg/test/dockerutil/exec.go
index 4c739c9e9..bf968acec 100644
--- a/pkg/test/dockerutil/exec.go
+++ b/pkg/test/dockerutil/exec.go
@@ -77,11 +77,6 @@ func (c *Container) doExec(ctx context.Context, r ExecOpts, args []string) (Proc
return Process{}, fmt.Errorf("exec attach failed with err: %v", err)
}
- if err := c.client.ContainerExecStart(ctx, resp.ID, types.ExecStartCheck{}); err != nil {
- hijack.Close()
- return Process{}, fmt.Errorf("exec start failed with err: %v", err)
- }
-
return Process{
container: c,
execid: resp.ID,