From: John Griffith <john.griffith8@gmail.com>
Date: Fri, 29 May 2015 19:02:13 +0000 (-0600)
Subject: Remove pretty_tox and use ostestr
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=68b4aad405c019d660b86f62617394e8b170c2ea;p=openstack-build%2Fcinder-build.git

Remove pretty_tox and use ostestr

This patch dumps pretty_tox.sh and gets Cinder on board
with using ostestr.  We're setting a concurrency value
of 6 (instances in the gate are 6 VCPU's), and things
seem to be stable at this value.

For folks running on 'smaller' system, it's still possible
to run in parallell; > ncpu's but if there are problems you
can still specify concurrency in your tox argument.  Tox will
take the last occurence of the conncurrency argument; so for
example if you specify "tox -epy27 -- --concurrency=4" the
interpretted value for concurrency in the run will be 4.

Closes-Bug: #1459852

Change-Id: Iae12b42e915da22c863781015512df4f30aa95ca
---

diff --git a/test-requirements.txt b/test-requirements.txt
index 35a420e76..37f120da3 100644
--- a/test-requirements.txt
+++ b/test-requirements.txt
@@ -18,4 +18,5 @@ python-subunit>=0.0.18
 testtools>=0.9.36,!=1.2.0
 testrepository>=0.0.18
 oslosphinx>=2.5.0  # Apache-2.0
+os-testr>=0.1.0
 tempest-lib>=0.5.0
diff --git a/tools/pretty_tox.sh b/tools/pretty_tox.sh
deleted file mode 100755
index c6ed23bac..000000000
--- a/tools/pretty_tox.sh
+++ /dev/null
@@ -1,16 +0,0 @@
-#!/usr/bin/env bash
-
-set -o pipefail
-
-TESTRARGS=$1
-
-# --until-failure is not compatible with --subunit see:
-#
-# https://bugs.launchpad.net/testrepository/+bug/1411804
-#
-# this work around exists until that is addressed
-if [[ "$TESTARGS" =~ "until-failure" ]]; then
-    python setup.py test --no-parallel --slowest --testr-args="$TESTRARGS"
-else
-    python setup.py test --no-parallel --slowest --testr-args="--subunit $TESTRARGS" | subunit-trace -f
-fi
diff --git a/tox.ini b/tox.ini
index 3060c8ad8..bcb1a9e6b 100644
--- a/tox.ini
+++ b/tox.ini
@@ -13,8 +13,14 @@ install_command = pip install {opts} {packages}
 
 deps = -r{toxinidir}/requirements.txt
        -r{toxinidir}/test-requirements.txt
-commands =
-  bash tools/pretty_tox.sh '{posargs}'
+
+# Note, we set concurrency to 6 here as a default
+# ostestr will take the last entry for occurence
+# so if you want to increase/decrease this you
+# can simply add concurrency arg to your tox
+# call ie: 'tox -epy27 -- --concurrency=4'
+commands = ostestr --concurrency=6 {posargs}
+
 whitelist_externals = bash
 
 [tox:jenkins]