diff options
| author | Ben Nemec <bnemec@redhat.com> | 2013-11-27 17:41:25 +0000 |
|---|---|---|
| committer | Ben Nemec <bnemec@redhat.com> | 2013-12-02 18:28:02 +0000 |
| commit | 4c47d3e759f15f1f3b230323ca723fbb1dd90a4f (patch) | |
| tree | a840174f80479c905edf5a94789a552b6b171fe5 /tox.ini | |
| parent | a957fee61828fd05795191b2c5d87e6dc57ff35b (diff) | |
| download | oslo-incubator-4c47d3e759f15f1f3b230323ca723fbb1dd90a4f.tar.gz | |
Fix locking in migration tests
The migration test code included an external lock with a hard-coded
relative path that breaks when run in Jenkins. It wasn't being hit
in Oslo because it was only locking for the postgres code path,
which isn't currently tested in Oslo.
This change moves the lock so that it applies to all db types and
removes the relative path. For the locks to work properly it was
also necessary to add the lockutils wrapper to the Oslo tox.ini.
Closes-Bug: 1255608
Change-Id: I333b833022f208e48e3b7e82e8cfad2764be5ea0
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -12,7 +12,8 @@ deps = -r{toxinidir}/requirements.txt -r{toxinidir}/test-requirements.txt commands = # due to dependencies between tests (bug 1192207) we use `--concurrency=1` option - python setup.py test --slowest --testr-args='--concurrency=1 {posargs}' + # Use the lockutils wrapper to ensure that external locking works correctly + python -m openstack.common.lockutils python setup.py test --slowest --testr-args='--concurrency=1 {posargs}' {toxinidir}/tools/config/generate_sample.sh -p openstack [flake8] @@ -32,7 +33,8 @@ commands = python ./tools/lint.py ./openstack setenv = VIRTUAL_ENV={envdir} commands = # due to dependencies between tests (bug 1192207) we use `--concurrency=1` option - python setup.py test --coverage --testr-args='--concurrency=1 {posargs}' + # Use the lockutils wrapper to ensure that external locking works correctly + python -m openstack.common.lockutils python setup.py test --coverage --testr-args='--concurrency=1 {posargs}' [testenv:venv] commands = {posargs} |