diff options
| author | Zeling Feng <zeling@google.com> | 2020-11-30 21:21:21 -0800 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2020-11-30 21:22:55 -0800 |
| commit | 59a2c785bfba56e2ffb04083c213fa2ad23a36a2 (patch) | |
| tree | e9c3de0127930f75d693cdf981f7b87fae23d123 | |
| parent | 79e2364933bbd34dceb0b9025e4e7d145fdf4492 (diff) | |
Do not start a ContainerExec twice
ContainerExecStart and ContainerExecAttach both call the /exec/id/start API
endpoint.
PiperOrigin-RevId: 344946627
| -rw-r--r-- | pkg/test/dockerutil/exec.go | 5 |
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, |