django-health-check icon indicating copy to clipboard operation
django-health-check copied to clipboard

Allow CeleryPingHealthCheck to check user-specific queues

Open BoPeng opened this issue 1 year ago • 2 comments

CeleryPingHealthCheck checks if some pre-defined queues have active workers

https://github.com/revsys/django-health-check/blob/251b1562c1a7b4bb95b46066fbde0bfaac41567b/health_check/contrib/celery_ping/backends.py#L53

Upon checking, we found that the CELERY_QUEUES setting is defined but is empty for our production server. Because this setting is now called task_queues and its documentation suggests against specifying this setting and use automatic routing, we do not want to mess with this option for our server. Can django-health-check allow a setting for a list of queues to check?

BoPeng avatar May 08 '24 14:05 BoPeng

This is related to #374.

sevdog avatar May 14 '24 14:05 sevdog

Yes, but I was expecting a separate setting that allows us to bypass celery configuration since app.conf.CELERY_QUEUES is empty if auto-routing is used. I mean, the production server should be flexible enough to dispatch to available workers automatically but healthcheck should alert us when some workers and/or queues are unavailable.

BoPeng avatar May 14 '24 14:05 BoPeng