openinfra/refstack-client
Code Issues Proposed changes

Set concurrency to 0

Browse Source 
until refstack-client uses tempest tag which contains
default value for concurrency it needs to be set here.

Change-Id: Ic33639f39a21d5827c9ffe14fc76a47a36c7b47f
This commit is contained in:
Martin Kopec
2019-03-06 22:50:25 +00:00
parent 5f95355189
commit 5be0ac7a6f
2 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
refstack_client/refstack_client.py
View File

@@ -523,6 +523,13 @@ class RefstackClient:
cmd = [wrapper, 'tempest', 'run']
if not self.args.parallel:
cmd.append('--serial')
# TODO(mkopec) until refstack-client uses tempest tag which contains
# the following change https://review.openstack.org/#/c/641349/
# let's hardcode concurrency here, when the change is merged, the
# value of concurrency will be set as default in tempest so the
# following two lines can be deleted
cmd.append('--concurrency')
cmd.append('0')
# If a test list was specified, have it take precedence.
if self.args.test_list:
self.logger.info("Normalizing test list...")

15
refstack_client/tests/unit/test_client.py
View File

@@ -573,7 +573,8 @@ class TestRefstackClient(unittest.TestCase):
mock_popen.assert_called_with(
['%s/tools/with_venv.sh' % self.test_path, 'tempest', 'run',
'--serial', '--regex', 'tempest.api.compute'],
'--serial', '--concurrency', '0', '--regex',
'tempest.api.compute'],
stderr=None
)
@@ -604,7 +605,8 @@ class TestRefstackClient(unittest.TestCase):
client.test()
mock_popen.assert_called_with(
['%s/tools/with_venv.sh' % self.test_path, 'tempest', 'run',
'--serial', '--regex', 'tempest.api.compute'],
'--serial', '--concurrency', '0', '--regex',
'tempest.api.compute'],
stderr=None
)
@@ -637,7 +639,8 @@ class TestRefstackClient(unittest.TestCase):
client.test()
mock_popen.assert_called_with(
['%s/tools/with_venv.sh' % self.test_path, 'tempest', 'run',
'--serial', '--regex', 'tempest.api.compute'],
'--serial', '--concurrency', '0', '--regex',
'tempest.api.compute'],
stderr=None
)
@@ -678,7 +681,8 @@ class TestRefstackClient(unittest.TestCase):
mock_whitelist.assert_called_with('test-list.txt')
mock_popen.assert_called_with(
['%s/tools/with_venv.sh' % self.test_path, 'tempest', 'run',
'--serial', '--whitelist_file', '/tmp/some-list'],
'--serial', '--concurrency', '0', '--whitelist_file',
'/tmp/some-list'],
stderr=None
)
@@ -738,7 +742,8 @@ class TestRefstackClient(unittest.TestCase):
mock_popen.assert_called_with(
['%s/tools/with_venv.sh' % self.test_path, 'tempest', 'run',
'--serial', '--regex', 'tempest.api.compute'],
'--serial', '--concurrency', '0', '--regex',
'tempest.api.compute'],
stderr=None
)
# Since '1' is in the next-stream file, we expect the JSON output file