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

fix: bump minCapacity from 15 => 30 #8724

Merged
merged 2 commits into from
Sep 1, 2023
Merged

fix: bump minCapacity from 15 => 30 #8724

merged 2 commits into from
Sep 1, 2023

Conversation

jamesgorrie
Copy link
Contributor

What does this change?

Bumps the min capacity from 15-30 as it seems the minimum isn't holding up with all the new traffic we're sending it.

This is a hotfix until we split our stacks, and can start setting scaling policies and capacities for specifically for each of our stacks.

@jamesgorrie jamesgorrie requested a review from a team as a code owner September 1, 2023 10:31
@jamesgorrie jamesgorrie changed the title fix: bump minCapacity from 30 => 60 fix: bump minCapacity from 15 => 30 Sep 1, 2023
@github-actions
Copy link

github-actions bot commented Sep 1, 2023

Size Change: 0 B 🆕

Total Size: 0 B

compressed-size-action

@jamesgorrie jamesgorrie added the run_chromatic Runs chromatic when label is applied label Sep 1, 2023
@jamesgorrie jamesgorrie merged commit f146288 into main Sep 1, 2023
@jamesgorrie jamesgorrie deleted the 30-60 branch September 1, 2023 11:01
@jamesgorrie
Copy link
Contributor Author

Screenshot 2023-09-01 at 12 35 22

And we're live

ioannakok added a commit that referenced this pull request Oct 31, 2023
`minCapacity` was bumped from 15 to 30 in #8724. After the fronts migration was completed, it seemed that 15 instances weren't holding up with the new traffic we were sending. That was a hotfix and there is upcoming work to split our stacks (#8351) but until we reach this point we would like to see whether we could handle the traffic with 25 instances. This change is part of the research we're doing to optimise how we scale: #9322.

Co-authored-by: George B <[email protected]>
Co-authored-by: Ravi <[email protected]>
ioannakok added a commit that referenced this pull request Oct 31, 2023
`minCapacity` was bumped from 15 to 30 in #8724. After the fronts migration was completed, it seemed that 15 instances weren't holding up with the new traffic we were sending. That was a hotfix and there is upcoming work to split our stacks (#8351) but until we reach this point we would like to see whether we could handle the traffic with 25 instances. This change is part of the research we're doing to optimise how we scale: #9322.

Co-authored-by: George B <[email protected]>
Co-authored-by: Ravi <[email protected]>
ioannakok added a commit that referenced this pull request Nov 1, 2023
`minCapacity` was bumped from 15 to 30 in #8724. After the fronts migration was completed, it seemed that 15 instances weren't holding up with the new traffic we were sending. That was a hotfix and there is upcoming work to split our stacks (#8351) but until we reach this point we would like to see whether we could handle the traffic with 25 instances. This change is part of the research we're doing to optimise how we scale: #9322.

Co-authored-by: George B <[email protected]>
Co-authored-by: Ravi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dotcom-rendering run_chromatic Runs chromatic when label is applied
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants