Error: backend does not implement forget about django-health-check HOT 9 CLOSED

Do you have a little more information for me?

from django-health-check.

It seems that health_check_celery3 calls 'forget' https://github.com/KristianOellegaard/django-health-check/blob/master/health_check_celery3/plugin_health_check.py#L30, which isn't supported by all backends. This results in an error even when Celery is working perfectly fine.

from django-health-check.

@samuelfekete looks like a bug to me. Do you have a solution for it?

from django-health-check.

Contributions are always welcome ;)

from django-health-check.

If not all backends support forget, then perhaps result.forget() should be surrounded with a try... except NotImplementedError, which can be ignored.

from django-health-check.

@samuelfekete if you put that into a pull-request and add a test, I'll merge it right away 😉

from django-health-check.

Here's the pull-request: #44
@codingjoe if you add tests for the celery3 health check, I'll add a test for this case ;)

from django-health-check.

@samuelfekete that is a good point. I know it's not nice to ask for tests, if there none to extend.
I'll try to make it happen soon.

from django-health-check.

I think this can be closed too @codingjoe.

from django-health-check.