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

[Issue #2600] scale up, memory scaling, operational docs #2720

Merged
merged 12 commits into from
Nov 12, 2024
Merged

[Issue #2600] scale up, memory scaling, operational docs #2720

merged 12 commits into from
Nov 12, 2024

Conversation

coilysiren
Copy link
Collaborator

@coilysiren coilysiren commented Nov 4, 2024 •
edited
Loading

Summary

Fixes #2600

Time to review: 5 mins

Changes proposed

  • Adds scaling docs. Half in OPERATIONS.md and half inside the terraform files
  • [ API, frontend ] x [ dev, staging, prod ] up to calculated values
  • Adds memory scaling in addition to the existing CPU scaling
  • Makes scale out more aggressive (300 seconds => 60 seconds)

Context for reviewers

This should be enough for a week or two. I'll re-evaluate prod weekly as we send out emails advertising the new system.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 5, 2024
@coilysiren coilysiren changed the title [Issue #2600] Scale up, swap autoscaling to memory [Issue #2600] Scale up, add memory scaling Nov 5, 2024
@coilysiren coilysiren changed the title [Issue #2600] Scale up, add memory scaling [Issue #2600] Scale up, add memory scaling, adds operational docs Nov 5, 2024
@coilysiren coilysiren changed the title [Issue #2600] Scale up, add memory scaling, adds operational docs [Issue #2600] Scale up, add memory scaling, operational docs Nov 5, 2024
@coilysiren coilysiren changed the title [Issue #2600] Scale up, add memory scaling, operational docs [Issue #2600] scale up, memory scaling, operational docs Nov 5, 2024
@coilysiren coilysiren marked this pull request as ready for review November 7, 2024 17:43
This was referenced Nov 7, 2024
Closed
Open
chouinar
chouinar approved these changes Nov 12, 2024
View reviewed changes
Copy link
Collaborator

@chouinar chouinar left a comment

Choose a reason for hiding this comment

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

LGTM as far as the configured numbers go

@coilysiren coilysiren merged commit 306d118 into main Nov 12, 2024
6 of 7 checks passed
@coilysiren coilysiren deleted the kai/scaling branch November 12, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chouinar chouinar chouinar approved these changes

@acouch acouch Awaiting requested review from acouch acouch is a code owner

@mdragon mdragon Awaiting requested review from mdragon mdragon is a code owner

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation infra terraform
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Scale web, OpenSearch, Postgres to handle 3-5x traffic level seen to date.
2 participants
@coilysiren @chouinar