diff options
| author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2012-02-19 12:13:48 +0100 |
|---|---|---|
| committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2012-02-27 23:16:08 +0100 |
| commit | baae1578dae4ccf6538baa1f362409e16f94f266 (patch) | |
| tree | dabee8159983bcdd814fc648ac3c4c3433e1292c /mediagoblin/processing.py | |
| parent | 34344f1d85be72741a9421a8a3af12ced35bbe35 (diff) | |
| download | mediagoblin-baae1578dae4ccf6538baa1f362409e16f94f266.tar.lz mediagoblin-baae1578dae4ccf6538baa1f362409e16f94f266.tar.xz mediagoblin-baae1578dae4ccf6538baa1f362409e16f94f266.zip | |
Let mark_entry_failed log unknown exceptions
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.
Diffstat (limited to 'mediagoblin/processing.py')
| -rw-r--r-- | mediagoblin/processing.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mediagoblin/processing.py b/mediagoblin/processing.py index 9e57380d..a07de869 100644 --- a/mediagoblin/processing.py +++ b/mediagoblin/processing.py @@ -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 |