Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow CeleryPingHealthCheck to check user-specific queues #428

Open
BoPeng opened this issue May 8, 2024 · 2 comments
Open

Allow CeleryPingHealthCheck to check user-specific queues #428

BoPeng opened this issue May 8, 2024 · 2 comments

Comments

@BoPeng
Copy link

BoPeng commented May 8, 2024

CeleryPingHealthCheck checks if some pre-defined queues have active workers

defined_queues = app.conf.CELERY_QUEUES

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?

@sevdog
Copy link

sevdog commented May 14, 2024

This is related to #374.

@BoPeng
Copy link
Author

BoPeng commented May 14, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants