-
Notifications
You must be signed in to change notification settings - Fork 0
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
Move email delivery into their own dedicated queues #206
Labels
Throughput
Will boost our throughput
Comments
sastels
changed the title
Copy of Move SMS into their own dedicated queues
Move email delivery into their own dedicated queues
Oct 10, 2023
PR to use the new queues in celery: |
PR to add alarms for the new queues |
PR to add the queues to the email dashboard |
These PRs can wait to be merged / released after I'm back (Oct 23) |
Jimmy approved and improved the PRs! To be merged today after am release |
in production |
Looks like it's working 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
As a Notify developer, I need email delivery tasks to be in their own queues so that I can treat them differently than other tasks (ex database, callbacks).
WHY are we building?
Moving email delivery to their own queues will allow us to scale up email delivery without scaling up other tasks such as database operations or callback processing. This will also allow better monitoring and alarming for email delivery issues.
WHAT are we building?
Move email delivery tasks to their own dedicated celery queues. Note that this task does not include moving the email delivery tasks to their own dedicated pool of celery workers.
VALUE created by our solution
We will be able to scale up email delivery without scaling up other parts of the system.
Acceptance Criteria
deliver_email()
tasks go through their own celery queues.QA Steps
The text was updated successfully, but these errors were encountered: