From 2e7e5b65e7580870eccd337d1d57fcf4d42bdd85 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=F0=9F=87=BA=F0=9F=87=A6=20Sviatoslav=20Sydorenko=20=28?= =?UTF-8?q?=D0=A1=D0=B2=D1=8F=D1=82=D0=BE=D1=81=D0=BB=D0=B0=D0=B2=20=D0=A1?= =?UTF-8?q?=D0=B8=D0=B4=D0=BE=D1=80=D0=B5=D0=BD=D0=BA=D0=BE=29?= Date: Thu, 13 Mar 2025 12:24:37 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=AA=20Drop=20venv=20when=20making=20co?= =?UTF-8?q?verage=20XML=20@=20CI?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This deletes the fallback that would use `--venv` when `--docker` is unavailable in `ansible-test`. But this option is present in all supported versions of Ansible as it was added in v2.12 which is EOL already. PR #84819 --- .azure-pipelines/scripts/report-coverage.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.azure-pipelines/scripts/report-coverage.sh b/.azure-pipelines/scripts/report-coverage.sh index 4db905eae28..edf7580ad6d 100755 --- a/.azure-pipelines/scripts/report-coverage.sh +++ b/.azure-pipelines/scripts/report-coverage.sh @@ -12,6 +12,6 @@ if ! ansible-test --help >/dev/null 2>&1; then pip install https://github.com/ansible/ansible/archive/devel.tar.gz --disable-pip-version-check fi -# Generate stubs using docker (if supported) otherwise fall back to using a virtual environment instead. -# The use of docker is required when Powershell code is present, but Ansible 2.12 was the first version to support --docker with coverage. -ansible-test coverage xml --group-by command --stub --docker --color -v || ansible-test coverage xml --group-by command --stub --venv --color -v +# Generate stubs using docker. +# The use of docker is mandatory when Powershell code is present. +ansible-test coverage xml --group-by command --stub --docker --color -v