Merge lp:~nuclearbob/qlbr/rtm-cache-clear into lp:~canonical-platform-qa/qlbr/rtm-reports

[Max Brustkern]

This branch clears the django cache when a data download is finished.

[Max Brustkern (nuclearbob)]

I also added a command to clear the cache for use when updates occur in the charm.

[Allan LeSage (allanlesage)]

Forgive me but aren't these some relevant settings to set up this caching system before being able to clear anything?

review: Needs Information

[Max Brustkern (nuclearbob)]

Yep, we've already got those here:
http://bazaar.launchpad.net/~canonical-platform-qa/qlbr/rtm-reports/view/head:/qlbr/settings.py#L194

At some point I'd like to clean this up and get it back into the main branch, and as part of that, I think I'd like to move to using a dummy cache by default so that installations (i.e. via the charm) can add their own, but right now I don't want to mess with that so we can keep the same code on the instance. Hopefully the instance will be replaced soon.

[Allan LeSage (allanlesage)]

Thanks Max, I missed the merge-into bit above, I can see how this would be necessary, LGTM.

review: Approve

=== modified file 'qlbr/management/commands/get_task_data.py'
--- qlbr/management/commands/get_task_data.py	2014-09-25 19:57:35 +0000
+++ qlbr/management/commands/get_task_data.py	2014-10-03 15:47:53 +0000
@@ -19,6 +19,7 @@
 import logging
 from optparse import make_option
 
+from django.core.cache import cache
 from django.core.exceptions import ObjectDoesNotExist
 from django.core.management.base import BaseCommand
 
@@ -238,3 +239,4 @@
                 'QA Reports', 'production', CACHEDIR,
                 credentials_file=options['credentials_file'])
         self.search_tasks()
+        cache.clear()