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

Remove django-celery dependency from active_status endpoint #7

Open
psychok7 opened this issue Aug 8, 2018 · 0 comments
Open

Remove django-celery dependency from active_status endpoint #7

psychok7 opened this issue Aug 8, 2018 · 0 comments

Comments

@psychok7
Copy link
Owner

psychok7 commented Aug 8, 2018

https://github.com/celery/django-celery is deprecated and it is also an unnecessary dependency.

We can redo the active_status endpoint without django-celery but using https://github.com/jezdez/django-celery-monitor/

So the new code would be added here: https://github.com/psychok7/django-celery-inspect/blob/master/celery_inspect/api/viewsets.py#L29 and it would instead import from:

from django_celery_monitor.models import WorkerState (and not from djcelery.models import WorkerState) and the check of installed apps would be django_celery_monitor instead of djcelery

@psychok7 psychok7 changed the title Try to remove django-celery dependency from active_status endpoint Remove django-celery dependency from active_status endpoint Aug 8, 2018
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

1 participant