]> git.apps.os.sepia.ceph.com Git - ceph-build.git/commitdiff
cephadm-ansible-prs: make tox 'not run everything' cephadm-ansible-prs-fix 2002/head
authorGuillaume Abrioux <gabrioux@redhat.com>
Wed, 18 May 2022 09:15:58 +0000 (11:15 +0200)
committerGuillaume Abrioux <gabrioux@redhat.com>
Wed, 18 May 2022 09:15:58 +0000 (11:15 +0200)
This makes tox run each environment explicitly one after the other.
This way if an environment fails, tox stops its execution.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
cephadm-ansible-prs/build/build

index cb570e9bcdb4049104af5bfbf806af8e8287f172..e78a23ea1484160e60b8cd8fa68e8cf02f7d6739 100644 (file)
@@ -1,14 +1,16 @@
 #!/bin/bash
 
 # the following two methods exist in scripts/build_utils.sh
+# shellcheck disable=SC2034
 pkgs=( "tox" )
 TEMPVENV=$(create_venv_dir)
 VENV=${TEMPVENV}/bin
 set_centos_python3_version "python3.9"
-install_python_packages $TEMPVENV "pkgs[@]" "pip==22.0.4"
+install_python_packages "$TEMPVENV" "pkgs[@]" "pip==22.0.4"
 
 # XXX this might not be needed
-source $VENV/activate
+# shellcheck source=/dev/null
+source "${VENV}"/activate
 
 WORKDIR=$(mktemp -td tox.XXXXXXXXXX)
 
@@ -19,4 +21,9 @@ update_vagrant_boxes
 
 rm -rf "${HOME}"/ansible/facts/*
 
-"${VENV}"/tox --workdir="${TEMPVENV}" -c tox.ini -r -v -- --provider=libvirt
+tox_envs="$("$VENV"/tox -l)"
+
+for tox_env in "${tox_envs[@]}"
+do
+  "${VENV}"/tox --workdir="${TEMPVENV}" -c tox.ini -r -v -e "${tox_env}" -- --provider=libvirt
+done