don't include stderr in moby exec output

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
This commit is contained in:
Nicolas De Loof 2021-04-09 15:09:48 +02:00
parent 76cd14a85d
commit 4916b36b99
No known key found for this signature in database
GPG Key ID: 9858809D6F8F6E7E
2 changed files with 4 additions and 4 deletions

View File

@ -136,5 +136,6 @@ func ExecSilent(ctx context.Context, args ...string) ([]byte, error) {
args = os.Args[1:] args = os.Args[1:]
} }
cmd := exec.CommandContext(ctx, ComDockerCli, args...) cmd := exec.CommandContext(ctx, ComDockerCli, args...)
return cmd.CombinedOutput() cmd.Stderr = os.Stderr
return cmd.Output()
} }

View File

@ -398,8 +398,7 @@ func TestLegacy(t *testing.T) {
t.Run("host flag", func(t *testing.T) { t.Run("host flag", func(t *testing.T) {
res := c.RunDockerOrExitError("-H", "tcp://nonexistent:123", "version") res := c.RunDockerOrExitError("-H", "tcp://nonexistent:123", "version")
assert.Assert(t, res.ExitCode == 1) assert.Assert(t, res.ExitCode == 1)
assert.Assert(t, strings.Contains(res.Stdout(), "dial tcp: lookup nonexistent"), res.Stdout()) assert.Assert(t, strings.Contains(res.Stderr(), "dial tcp: lookup nonexistent"), res.Stderr())
}) })
t.Run("remote engine context", func(t *testing.T) { t.Run("remote engine context", func(t *testing.T) {
@ -408,7 +407,7 @@ func TestLegacy(t *testing.T) {
res := c.RunDockerOrExitError("version") res := c.RunDockerOrExitError("version")
assert.Assert(t, res.ExitCode == 1) assert.Assert(t, res.ExitCode == 1)
assert.Assert(t, strings.Contains(res.Stdout(), "dial tcp: lookup nonexistent"), res.Stdout()) assert.Assert(t, strings.Contains(res.Stderr(), "dial tcp: lookup nonexistent"), res.Stderr())
}) })
t.Run("existing contexts delegate", func(t *testing.T) { t.Run("existing contexts delegate", func(t *testing.T) {