Skip to content

docs: update aws instance recommendations #17344

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

Merged
merged 2 commits into from
Apr 10, 2025
Merged

docs: update aws instance recommendations #17344

merged 2 commits into from
Apr 10, 2025

Conversation

EdwardAngert
Copy link
Contributor

@EdwardAngert EdwardAngert commented Apr 10, 2025

from @jatcod3r on Slack:

for the AWS recs on our validated arch docs, should we be referencing customers to use non-T type instances?
Once you've exceeded EC2's CPU credits Coder starts performing poorly.
We do suggest to scale for peak demand, so does recommending something from the cpu or memory optimized types make sense?

preview

@EdwardAngert EdwardAngert added the docs Area: coder.com/docs label Apr 10, 2025
@EdwardAngert EdwardAngert requested a review from jatcod3r April 10, 2025 18:13
@EdwardAngert EdwardAngert self-assigned this Apr 10, 2025
Copy link
Contributor

@jatcod3r jatcod3r 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 to me! Also thanks adding in the index below explaining the AWS instance types, it's really helpful!

Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

Looks Great.

@EdwardAngert EdwardAngert merged commit e5ba8b7 into main Apr 10, 2025
33 checks passed
@EdwardAngert EdwardAngert deleted the aws-ec2-arch branch April 10, 2025 18:35
@github-actions github-actions bot locked and limited conversation to collaborators Apr 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Area: coder.com/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants