plugin/raven: Fix paster's celery config issue
authorJoar Wandborg <joar@wandborg.se>
Sat, 23 Feb 2013 23:23:05 +0000 (00:23 +0100)
committerJoar Wandborg <joar@wandborg.se>
Sat, 2 Mar 2013 23:46:05 +0000 (00:46 +0100)
Check for CELERY_CONFIG_MODULE before we import raven.contrib.celery. It
seems that the import otherwise sets up the celery client before we get
to pass it our mediagoblin-specific settings.

mediagoblin/plugins/raven/__init__.py

index a0e83c0aa98ea21846c80a2ad5940038eed604c9..16b3b94b88364a0b7824ea64898d7d6cfff397dd 100644 (file)
@@ -18,13 +18,17 @@ import os
 import logging
 
 from mediagoblin.tools import pluginapi
-from raven import Client
-from raven.contrib.celery import register_signal
 
 _log = logging.getLogger(__name__)
 
 
 def setup_plugin():
+    if not os.environ.get('CELERY_CONFIG_MODULE'):
+       # Exit early if we're (seemingly) not called from the celery process
+        return
+
+    from raven import Client
+    from raven.contrib.celery import register_signal
     config = pluginapi.get_config('mediagoblin.plugins.raven')
 
     _log.info('Setting up raven for celery...')