oslo.db now requires testresources/testscenarios at runtime

Bug #1429233 reported by Matt Riedemann
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
oslo.db
Fix Released
Low
Matt Riedemann

Bug Description

Change https://review.openstack.org/#/c/120870/ moved the testresouces requirement from test-requirements* to requirements.txt, this means that any distro that wants to package and ship oslo.db >= 1.5.0 has to provide testresources at runtime too if they are matching what's in requirements.txt, which seems wrong.

Looking at what's using testresources outside of tests, it's the provision module:

https://review.openstack.org/#/c/120870/24/oslo_db/sqlalchemy/provision.py

And that's only imported in non-test code in oslo_db.sqlalchemy.utils.is_backend_avail which is deprecated, and that method is only called from other test code, so even though testresources is imported in provision which isn't under tests, it seems that testresources should still be under test-requirements since that's what it's used for, testing of multiple backends.

Revision history for this message
Matt Riedemann (mriedem) wrote :
summary: - oslo.db now requires testresources at runtime
+ oslo.db now requires testresources/testscenarios at runtime
Changed in oslo.db:
status: New → In Progress
assignee: nobody → Matt Riedemann (mriedem)
Revision history for this message
Matt Riedemann (mriedem) wrote :
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

The discussion of the policy related to dependency management in the libraries is on the spec proposed in https://review.openstack.org/157135

Changed in oslo.db:
importance: Undecided → Wishlist
status: In Progress → Confirmed
Revision history for this message
Matt Riedemann (mriedem) wrote :
Changed in oslo.db:
status: Confirmed → In Progress
importance: Wishlist → Low
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :
Changed in oslo.db:
status: In Progress → Fix Committed
Changed in oslo.db:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.