Django cache machine 0.8 has some fixes for Django 1.4 and also for multi-db setups. I upgraded quickly, fixed a few backwards incompatible things and discovered there were quite a few test errors afterwards. Counts, e.g., seems to have changed. I found a few of these: eq_(qs.count(), old_cnt + 1) AssertionError: 0 != 1 eq_(EscalationQueue.objects.filter(addon=self.app).count(), 1) AssertionError: 0 != 1 etc. Work in progress patch on this branch: https://github.com/robhudson/zamboni/tree/upgrade-cache-machine
"How about delete django cache machine" -- ashort -> rob due to patch in progress. sry! :)
Assignee: nobody → robhudson.mozbugs
Priority: -- → P3
One thing to note, I think I might have discovered a bug in our version: no_cache() doesn't seem to working on count()s. Since there are changes related to counts in 0.8 it might be interesting to investigate further.
So, that bug with no_cache() and count()s has been fixed 2 years ago in cache-machine. Let's try to upgrade.
Assignee: robhudson.mozbugs → mpillard
This change is related to how we cache stuff and can affect all marketplace. So here are some generic STRs to test that we didn't break anything major with it: STR: - Load marketplace devhub, submit an app, see if everything looks ok in the process - Wait a couple minutes, load the reviewer tools, see if you can see your app in the queue - Approve the app, see if it becomes public on the marketplace like it should.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-10-07
Verified as fixed in https://marketplace-dev.allizom.org/ on FF 27 (Win 7). Postfix screencast http://screencast.com/t/RAwoYnUKw Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.