Closed Bug 792917 Opened 12 years ago Closed 12 years ago

Re-review queue breaks amo-master

Categories

(Marketplace Graveyard :: Code Quality, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2012-09-27

People

(Reporter: andy+bugzilla, Assigned: robhudson)

References

Details

(Whiteboard: [qa-])

Deleting an addon can AMO causes failures.

This is because Django collects all models (even uninstalled ones):

https://github.com/django/django/blob/master/django/db/models/options.py#L405

And then tries to delete the relevant tables. At some point mkt/reviewers is imported and so the revreviewqueue is registered as a table to be deleted. However the table does not exist on the test server. This causes failures like this:

https://ci.mozilla.org/job/amo-master/7376/testReport/junit/workspace.apps.addons.tests.test_models/TestAddonDelete/test_cascades/

This is a general problem with a model existing in one site (AMO) and the relevant review queue existing in another site (Marketplace).
See also bug 792916.
I can take this. I'll probably just move the RereviewQueue model to editor/models.py. I had to do this with the EscalationQueue already.
Assignee: nobody → robhudson.mozbugs
https://github.com/mozilla/zamboni/commit/59688be
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2012-09-27
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.