]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove the hardcoded concurrency limit for ostestr
authorJohn Griffith <john.griffith8@gmail.com>
Fri, 19 Jun 2015 18:32:01 +0000 (12:32 -0600)
committerJohn Griffith <john.griffith8@gmail.com>
Fri, 19 Jun 2015 18:34:34 +0000 (12:34 -0600)
ostestr by default uses concurrency = ncpu.

This caused all sorts of things to blow up in Cinder
when one got over 6 cpus.  Since the initial merge a
patch has landed that appears to fix the issues, and
we now run reliably at 8 and even 12 cpus.

This patch removes the hard coded concurrency=6 setting
from tox.ini

Change-Id: I483e53acaf1bb71deeba20d6c81184112cbcb046

tox.ini

diff --git a/tox.ini b/tox.ini
index c433451f1dfab34ee3726ecfdf1ac42779544653..ae06aa50ef28418383a2b1a2b8fce8afb05ed1c7 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -14,12 +14,11 @@ install_command = pip install {opts} {packages}
 deps = -r{toxinidir}/requirements.txt
        -r{toxinidir}/test-requirements.txt
 
-# 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
+# By default ostestr will set concurrency
+# to ncpu, to specify something else use
+# the concurrency=<n> option.
 # call ie: 'tox -epy27 -- --concurrency=4'
-commands = ostestr --concurrency=6 {posargs}
+commands = ostestr {posargs}
 
 whitelist_externals = bash
 passenv = http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY