-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add Celery #207
Comments
We keep talking about adding Celery, and we should probably bite the bullet and do it. Being able to schedule background jobs will have other uses we should probably document here. |
@jerbob I defer to you here, but presumably this means we're going to need to add another container? We need to consider how that works with self-hosted instances, perhaps a job for the RACTF public address system? Would probably involve us having to write a migration tool for Docker Compose files into ractl... I don't really want to do the hacky solution of running this as a background process in the |
Ye s to cobfirm we need another container |
The same way we used to start two instances for ASGI+WSGI |
We still do |
no no nonono wait wait wait wait wait |
wait why are we still starting two containers? |
If we're going to change this, it should probably be its own issue |
I think we fixed that bug in app-restructure |
Doesn't look like it, docker-compose.yml still has sockets and web |
Okay, we should only need a single container to handle HTTP + websockets but I suppose that's out of scope for this ticket. |
I have been fortunate to have used celery as part of previous employment in the past two years, and I have come to the conclusion that we must not, under any circumstances, use celery. I can provide reasons if interested. |
We need to add celery
The text was updated successfully, but these errors were encountered: