removal of testresources from setup.cfg has broken this dependency for oslo.db 4.34
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
oslo.db |
Fix Released
|
Undecided
|
Mike Bayer |
Bug Description
this is an issue that keeps coming up, for example at https:/
oslo.db is in this funny position where it is not only a library that other projects use for database access, it is also a library that is used to support running test suites. And as such, it includes modules within its normal library functions that themselves are dependencies on test-oriented libraries like testresources and testscenarios.
This implies that oslo.db really needs to have testresources and testscenarios in its normal requirements.txt, but for whatever reason, they keep getting removed. For a long time they found themselves in setup.cfg within the "test" section which seemed to fix things but then in https:/
To illustrate the issue:
git clone git://git.
cd keystone
# remove oslo.db from upper-requirments so they get the latest (4.34, not 4.33)
curl https:/
TOX_TESTENV_
output:
=======
Failures during discovery
=======
--- import errors ---
Failed to import test module: keystone.
Traceback (most recent call last):
File "/home/
module = self._get_
File "/home/
__import_
File "keystone/
from oslo_db.sqlalchemy import test_fixtures as db_fixtures
File "/home/
import testresources
ImportError: No module named testresources
Failed to import test module: keystone.
Traceback (most recent call last):
File "/home/
module = self._get_
File "/home/
__import_
File "keystone/
from oslo_db.sqlalchemy import test_fixtures as db_fixtures
File "/home/
import testresources
ImportError: No module named testresources
Failed to import test module: keystone.
Traceback (most recent call last):
File "/home/
module = self._get_
File "/home/
__import_
File "keystone/
from oslo_db.sqlalchemy import test_fixtures as db_fixtures
File "/home/
import testresources
ImportError: No module named testresources
solution:
testresources and testsceanrios need to be in requirements.txt, *not* test-requiremen
Fix proposed to branch: master /review. openstack. org/553801
Review: https:/