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

update: use separate worker service for each queue #235

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nathanfranklin
Copy link
Collaborator

@nathanfranklin nathanfranklin commented Jan 4, 2025

Overview:

This PR changes the local dev docker compose so that we have a single worker service for each queue. So now instead of one worker service running celery twice, we have a service for the heavy queue and a service for the default queue.

In #220, we added a heavy queue. Two comments at the time were about

Testing Steps:

  1. Confirm you can load an image (default queue) and import a point cloud (heavy queue)


stdin_open: true
tty: true
<<: *worker-base
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! New syntax we're using, should we link to documentation in a comment?
https://docs.docker.com/reference/compose-file/fragments/

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

Successfully merging this pull request may close these issues.

2 participants