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

12/11/2024 Production Deployment API hotfixes #1475

Merged
merged 7 commits into from
Dec 11, 2024
Merged

12/11/2024 Production Deployment API hotfixes #1475

merged 7 commits into from
Dec 11, 2024

Conversation

ccostino
Copy link
Contributor

A note to PR reviewers: it may be helpful to review our code review documentation to know what to keep in mind while reviewing pull requests.

Description

This is a production deployment for the API that includes a couple of hotfixes:

  • Adjusting the worker_max_tasks_per_child Celery config to 2000 (up from 500)
  • Adjusting the retry schedule to fire around every hour (give or take up to 20 minutes) and try for up to 72 hours to mirror AWS instead of just 4 hours

Security Considerations

  • These are small configuration adjustments to help improve application stability while under heavy use.

@ccostino ccostino added engineering python Pull requests that update Python code labels Dec 11, 2024
@ccostino ccostino self-assigned this Dec 11, 2024
@ccostino ccostino requested a review from ecayer December 11, 2024 14:44
Copy link
Contributor

@ecayer ecayer left a comment

Choose a reason for hiding this comment

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

Looks good as far as I can tell ;)

@stvnrlly stvnrlly merged commit fcfb6e6 into production Dec 11, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering python Pull requests that update Python code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants