From 32443f48fb85d9a03a355e4522ee5caf33d46660 Mon Sep 17 00:00:00 2001 From: Jon Skarpeteig Date: Thu, 17 Dec 2015 13:09:59 +0100 Subject: [PATCH] Remove docker debug output polluting the view With the -D flag present, the only extra output is of type: time="2015-12-17T10:50:24+01:00" level=debug msg="framesize: 74" between every single STDOUT line. This shouldn't be needed, as you would expect the debug flag to influence dox things, not Docker Change-Id: I224c9e13f5f2a0775afb0ca5f74dd7abf69adc45 --- dox/runner.py | 2 -- dox/tests/test_runner.py | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/dox/runner.py b/dox/runner.py index 926db7b..70b4d50 100644 --- a/dox/runner.py +++ b/dox/runner.py @@ -82,8 +82,6 @@ class Runner(object): def _docker_cmd(self, *args): base_docker = ['docker'] - if self.args.debug: - base_docker.append('-D') try: self._run_shell_command(base_docker + list(args)) except Exception as e: diff --git a/dox/tests/test_runner.py b/dox/tests/test_runner.py index aebec39..c227379 100644 --- a/dox/tests/test_runner.py +++ b/dox/tests/test_runner.py @@ -105,7 +105,7 @@ class TestRunner(base.TestCase): dr._run_shell_command = mock.MagicMock() dr._docker_cmd("version") dr._run_shell_command.assert_called_with( - ['docker', '-D', 'version'] + ['docker', 'version'] ) dr = doxrunner.Runner(argparse.Namespace(user_map=None,