Remove detox in favor of tox --parallel 16/16716/2
authorThanh Ha <zxiiro@gmail.com>
Tue, 27 Aug 2019 22:13:38 +0000 (18:13 -0400)
committerThanh Ha <zxiiro@gmail.com>
Tue, 27 Aug 2019 22:37:38 +0000 (18:37 -0400)
Tox officially supports parallel mode as of Tox 3.7 so drop the
usage of detox. Set parallel mode to 'auto' which will run as many
threads as available CPUs.

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: Id59ad10ed9b521a5c0ca3fef3e7e15053c2dc32c

shell/python-tools-install.sh
shell/tox-install.sh
shell/tox-run.sh

index a4bec33..c768653 100644 (file)
@@ -50,7 +50,7 @@ python-openstackclient~=3.16.0
 dogpile.cache~=0.6.8  # Version 0.7.[01] seems to break openstackclient
 more-itertools~=5.0.0
 niet~=1.4.2 # Extract values from yaml
-tox~=3.5.0
+tox>=3.7.0. # Tox 3.7 or greater is necessary for parallel mode support
 yq~=2.7.2
 EOF
 
index ad98336..4708e88 100644 (file)
@@ -15,7 +15,7 @@ echo "---> tox-install.sh"
 set -e -o pipefail
 
 # Tox version is pulled in through detox to mitigate version conflict
-python -m pip install --user --quiet --upgrade argparse detox tox-pyenv
+python -m pip install --user --quiet --upgrade tox-pyenv
 
 echo "----> pip freeze"
 pip freeze
index 90bd546..c159891 100644 (file)
@@ -34,15 +34,15 @@ set +e  # Allow detox to fail so that we can collect the logs in the next step
 PARALLEL="${PARALLEL:-true}"
 if [ "${PARALLEL}" = true ]; then
     if [ -n "$TOX_ENVS" ]; then
-        detox -e "$TOX_ENVS"  | tee -a "$ARCHIVE_TOX_DIR/detox.log"
+        tox -e "$TOX_ENVS" --parallel auto --parallel-live | tee -a "$ARCHIVE_TOX_DIR/tox.log"
         tox_status="${PIPESTATUS[0]}"
     else
-        detox | tee -a "$ARCHIVE_TOX_DIR/detox.log"
+        tox --parallel auto --parallel-live | tee -a "$ARCHIVE_TOX_DIR/tox.log"
         tox_status="${PIPESTATUS[0]}"
     fi
 else
     if [ -n "$TOX_ENVS" ]; then
-        tox -e "$TOX_ENVS"  | tee -a "$ARCHIVE_TOX_DIR/tox.log"
+        tox -e "$TOX_ENVS" | tee -a "$ARCHIVE_TOX_DIR/tox.log"
         tox_status="${PIPESTATUS[0]}"
     else
         tox | tee -a "$ARCHIVE_TOX_DIR/tox.log"