mitodl / django-server-status

Monitor server status for health checks.
GNU Affero General Public License v3.0
8 stars 0 forks source link

Catch all exceptions during celery requests. #22

Closed justinabrahms closed 8 years ago

justinabrahms commented 8 years ago

If there's an underlying redis error, we don't catch it and it 500s.

noisecapella commented 8 years ago

:+1:

Was there a specific exception that was uncaught before?

giocalitri commented 8 years ago

do you have any idea what are the other exceptions?

justinabrahms commented 8 years ago

connectionerror

Ultimately, it doesn't matter what the exceptions are. If it errors, we should be showing "DOWN"

noisecapella commented 8 years ago

No argument here

giocalitri commented 8 years ago

:+1: