diff options
| author | Boris Bobrov <breton@cynicmansion.ru> | 2014-08-11 20:59:35 +0400 |
|---|---|---|
| committer | Boris Bobrov <breton@cynicmansion.ru> | 2016-01-28 22:04:44 +0100 |
| commit | e3663c7be1d8c5a11b5e47050d921739a068b64f (patch) | |
| tree | 3c5cb86fe5627341eb927708afc7cd57498a53a8 /mediagoblin/processing | |
| parent | 4dd814a945e3cbb6c2e904844398223be9d86f49 (diff) | |
| download | mediagoblin-e3663c7be1d8c5a11b5e47050d921739a068b64f.tar.lz mediagoblin-e3663c7be1d8c5a11b5e47050d921739a068b64f.tar.xz mediagoblin-e3663c7be1d8c5a11b5e47050d921739a068b64f.zip | |
changed the way we save information about exceptions
Diffstat (limited to 'mediagoblin/processing')
| -rw-r--r-- | mediagoblin/processing/__init__.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/mediagoblin/processing/__init__.py b/mediagoblin/processing/__init__.py index aa8f1447..112b9d35 100644 --- a/mediagoblin/processing/__init__.py +++ b/mediagoblin/processing/__init__.py @@ -325,14 +325,12 @@ def mark_entry_failed(entry_id, exc): 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 - # here) + # Looks like no, let's record it so that admin could ask us about the + # reason atomic_update(mgg.database.MediaEntry, {'id': entry_id}, {u'state': u'failed', - u'fail_error': None, + u'fail_error': u'Unhandled exception: {0}'.format(unicode(exc)), u'fail_metadata': {}}) |