diff --git a/dev/archery/archery/docker/core.py b/dev/archery/archery/docker/core.py index 1c486e7aae629..2bc2a9939e4b9 100644 --- a/dev/archery/archery/docker/core.py +++ b/dev/archery/archery/docker/core.py @@ -232,7 +232,7 @@ def _execute_docker(self, *args, **kwargs): def pull(self, service_name, pull_leaf=True, ignore_pull_failures=True): def _pull(service): - args = ['pull'] + args = ['pull', '--quiet'] if service['image'] in self.pull_memory: return @@ -427,9 +427,9 @@ def run(self, service_name, command=None, *, env=None, volumes=None, def push(self, service_name, user=None, password=None): def _push(service): if self.config.using_docker: - return self._execute_docker('push', service['image']) + return self._execute_docker('push', '--quiet', service['image']) else: - return self._execute_compose('push', service['name']) + return self._execute_compose('push', '--quiet', service['name']) if user is not None: try: diff --git a/dev/archery/archery/docker/tests/test_docker.py b/dev/archery/archery/docker/tests/test_docker.py index 0849d1e97c984..5dd4b1bccecbe 100644 --- a/dev/archery/archery/docker/tests/test_docker.py +++ b/dev/archery/archery/docker/tests/test_docker.py @@ -270,7 +270,7 @@ def test_compose_default_params_and_env(arrow_compose_path): def test_forwarding_env_variables(arrow_compose_path): expected_calls = [ - "pull --ignore-pull-failures conda-cpp", + "pull --quiet --ignore-pull-failures conda-cpp", "build conda-cpp", ] expected_env = PartialEnv( @@ -290,24 +290,24 @@ def test_compose_pull(arrow_compose_path): compose = DockerCompose(arrow_compose_path) expected_calls = [ - "pull --ignore-pull-failures conda-cpp", + "pull --quiet --ignore-pull-failures conda-cpp", ] with assert_compose_calls(compose, expected_calls): compose.clear_pull_memory() compose.pull('conda-cpp') expected_calls = [ - "pull --ignore-pull-failures conda-cpp", - "pull --ignore-pull-failures conda-python", - "pull --ignore-pull-failures conda-python-pandas" + "pull --quiet --ignore-pull-failures conda-cpp", + "pull --quiet --ignore-pull-failures conda-python", + "pull --quiet --ignore-pull-failures conda-python-pandas" ] with assert_compose_calls(compose, expected_calls): compose.clear_pull_memory() compose.pull('conda-python-pandas') expected_calls = [ - "pull --ignore-pull-failures conda-cpp", - "pull --ignore-pull-failures conda-python", + "pull --quiet --ignore-pull-failures conda-cpp", + "pull --quiet --ignore-pull-failures conda-python", ] with assert_compose_calls(compose, expected_calls): compose.clear_pull_memory() @@ -316,8 +316,8 @@ def test_compose_pull(arrow_compose_path): def test_compose_pull_params(arrow_compose_path): expected_calls = [ - "pull --ignore-pull-failures conda-cpp", - "pull --ignore-pull-failures conda-python", + "pull --quiet --ignore-pull-failures conda-cpp", + "pull --quiet --ignore-pull-failures conda-python", ] compose = DockerCompose(arrow_compose_path, params=dict(UBUNTU='18.04')) expected_env = PartialEnv(PYTHON='3.8', PANDAS='latest') @@ -483,7 +483,7 @@ def test_compose_push(arrow_compose_path): for image in ["conda-cpp", "conda-python", "conda-python-pandas"]: expected_calls.append( mock.call(["docker", "compose", f"--file={compose.config.path}", - "push", image], check=True, env=expected_env) + "push", "--quiet", image], check=True, env=expected_env) ) with assert_subprocess_calls(expected_calls): compose.push('conda-python-pandas', user='user', password='pass') diff --git a/dev/archery/archery/docker/tests/test_docker_cli.py b/dev/archery/archery/docker/tests/test_docker_cli.py index c117a3edfff65..326147173d000 100644 --- a/dev/archery/archery/docker/tests/test_docker_cli.py +++ b/dev/archery/archery/docker/tests/test_docker_cli.py @@ -33,7 +33,7 @@ def test_docker_run_with_custom_command(run, build, pull): assert result.exit_code == 0 pull.assert_called_once_with( - "ubuntu-cpp", pull_leaf=True, + "--quiet", "ubuntu-cpp", pull_leaf=True, ) build.assert_called_once_with( "ubuntu-cpp", @@ -72,7 +72,7 @@ def test_docker_run_options(run, build, pull): result = CliRunner().invoke(docker, args) assert result.exit_code == 0 pull.assert_called_once_with( - "ubuntu-cpp", pull_leaf=True, + "--quiet", "ubuntu-cpp", pull_leaf=True, ) build.assert_called_once_with( "ubuntu-cpp", @@ -149,7 +149,7 @@ def test_docker_run_only_pulling_and_building(build, pull): result = CliRunner().invoke(docker, args) assert result.exit_code == 0 pull.assert_called_once_with( - "ubuntu-cpp", pull_leaf=True, + "--quiet", "ubuntu-cpp", pull_leaf=True, ) build.assert_called_once_with( "ubuntu-cpp",