Capture and properly handle errors.
authorChristopher Allan Webber <cwebber@dustycloud.org>
Sat, 13 Aug 2011 17:21:06 +0000 (12:21 -0500)
committerChristopher Allan Webber <cwebber@dustycloud.org>
Sat, 13 Aug 2011 17:21:06 +0000 (12:21 -0500)
commit6788b4123ef00241d6b6c80ca93d655e4307d6e3
tree83510de6a4c8fc0852e539196c82c5020a5d58ed
parent4a477e246d07a4c26f084db2596caf3310b78609
Capture and properly handle errors.

Handled in several places:

 - In the run() of the ProcessMedia itself for
   handled (BaseProcessingFail derived) errors (best to do these not
   in on_failure because the errors are highlighted in celeryd in a
   way that looks inappropriate for when the errors are well handled)
 - In ProcessMedia.on_failure() for all other errors
 - In the submit view where all exceptions are caught, media is marked
   at having failed, then the error is re-raised.  (The reason for
   this is that users running in "lazy" mode will get errors
   propagated by celery and so on_failure won't run for them.)
mediagoblin/process_media/__init__.py
mediagoblin/submit/views.py