Closed
Bug 1042169
Opened 10 years ago
Closed 10 years ago
django-multidb-router missing from requirements.txt
Categories
(Socorro :: Infra, task)
Tracking
(Not tracked)
RESOLVED
FIXED
95
People
(Reporter: bramwelt, Assigned: peterbe)
References
Details
Running `make bootstrap` in a clean Vagrant VM fails with: 'Improperly Configured: Error importing database router PinningMasterSlaveRouter: "No module named multidb"' This is resolved by pip installing 'django-multidb-router'. 'django-multidb-router' should be added to requirements.txt and pinned to what playdoh-lib included before 965589 was resolved.
Assignee | ||
Comment 1•10 years ago
|
||
That's strange. I need to look up what `make bootstrap` does differently. We don't need django-multidb-router and the bug is that it gets required by something.
Assignee | ||
Comment 2•10 years ago
|
||
This is the middleware class that could cause that error: multidb.middleware.PinningRouterMiddleware Basically, it should not by MIDDLEWARE_CLASSES.
Reporter | ||
Comment 3•10 years ago
|
||
multidb.middleware.PinningRouterMiddleware is not listed in any MIDDLEWARE_CLASSES. It is however required by funfactory, which we rely on.
Assignee | ||
Comment 4•10 years ago
|
||
My bad, some other thing is requiring multidb which we ought to get rid of.
Assignee: nobody → peterbe
Assignee | ||
Comment 5•10 years ago
|
||
PR https://github.com/mozilla/socorro/pull/2235
Reporter | ||
Comment 6•10 years ago
|
||
funfactory requires multidb and sets DATABASE_ROUTERS to 'multidb.PinningMasterSlaveRouter' https://github.com/mozilla/funfactory/blob/afaa4934297c79c98cc18b106ad73ff92ff74472/funfactory/settings_base.py#L31 Unsetting this (setting to an empty tuple) should fix the issue.
Comment 7•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/f27c3abfad6c25dd2f1c5af106b50f4012825bdd fixes bug 1042169 - unset DATABASE_ROUTERS in settings https://github.com/mozilla/socorro/commit/8e22e30fe002138d0f08d6b7db80a31e0064b9a6 Merge pull request #2235 from peterbe/bug-1042169-django-multidb-router-missing-from-requirementstxt fixes bug 1042169 - unset DATABASE_ROUTERS in settings
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Target Milestone: --- → 95
You need to log in
before you can comment on or make changes to this bug.
Description
•