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
This commit is contained in:
parent
aa5e8cfbe9
commit
68b4aad405
@ -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
|
||||
|
@ -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
|
10
tox.ini
10
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]
|
||||
|
Loading…
Reference in New Issue
Block a user