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.

Sign up for GitHub

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

Jump to bottom

Update celery worker memory #4890

Merged
merged 2 commits into from
Jun 16, 2021
Merged

Update celery worker memory #4890

merged 2 commits into from
Jun 16, 2021

Conversation

pkfec
Copy link
Contributor

@pkfec pkfec commented Jun 16, 2021 •
edited
Loading

Summary (required)

Allocate more memory to worker instances in Production to address worker out of memory issues happening more frequently in production.

(Include a summary of proposed changes and connect issue below)

  • Reduce worker memory to 512M in manifest_celery_worker_dev.yml
  • Increase worker memory to 1.5G manifest_celery_worker_prod.yml

Required reviewers

  • Any one backend developer.

Impacted areas of the application

  • Downloads, Scheduled tasks.

@pkfec pkfec requested review from lbeaufort and fec-jli June 16, 2021 15:21
@pkfec pkfec self-assigned this Jun 16, 2021
@pkfec pkfec added this to the Sprint 15.1 milestone Jun 16, 2021
lbeaufort
lbeaufort approved these changes Jun 16, 2021
View reviewed changes
Copy link
Member

@lbeaufort lbeaufort left a comment

Choose a reason for hiding this comment

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

@pkfec thanks for getting this together so quickly. Noting here that this is part of the work for #4864

@pkfec pkfec changed the title Feature/update celery worker memory Update celery worker memory Jun 16, 2021
fec-jli
fec-jli approved these changes Jun 16, 2021
View reviewed changes
Copy link
Contributor

@fec-jli fec-jli 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. Thanks

@lbeaufort lbeaufort merged commit c8779d0 into develop Jun 16, 2021
@pkfec pkfec mentioned this pull request Jun 16, 2021
Closed
@lbeaufort lbeaufort deleted the feature/update-celery-worker-memory branch June 22, 2021 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lbeaufort lbeaufort lbeaufort approved these changes

@fec-jli fec-jli fec-jli approved these changes

Assignees

@pkfec pkfec

Labels
Please review
Projects
None yet
Milestone
Sprint 15.1
Development

Successfully merging this pull request may close these issues.
3 participants
@pkfec @fec-jli @lbeaufort