Let mark_entry_failed log unknown exceptions
authorElrond <elrond+mediagoblin.org@samba-tng.org>
Sun, 19 Feb 2012 11:13:48 +0000 (12:13 +0100)
committerElrond <elrond+mediagoblin.org@samba-tng.org>
Mon, 27 Feb 2012 22:16:08 +0000 (23:16 +0100)
I don't know exactly why, but an exception during
processing hasn't found its way up. The entry was marked as
failed and that was it. So I decided to add a _log.warn to
the part of mark_entry_failed that handles general
exceptions.

mediagoblin/processing.py

index 9e57380dec8bc4941cdbd932b7cc74b9fe7b161c..a07de869fd1c178d96df5f79168888417854666d 100644 (file)
@@ -114,6 +114,7 @@ def mark_entry_failed(entry_id, exc):
                       u'fail_error': exc.exception_path,
                       u'fail_metadata': exc.metadata}})
     else:
+        _log.warn("No idea what happened here, but it failed: %r", exc)
         # Looks like no, so just mark it as failed and don't record a
         # failure_error (we'll assume it wasn't handled) and don't record
         # metadata (in fact overwrite it if somehow it had previous info