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

bump staging RDS instance to T3 Small #4081

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

freemvmt
Copy link
Contributor

@freemvmt freemvmt commented Dec 13, 2024

Related to this ticket.

This PR bumps the RDS PostgreSQL DB instance used on staging to db.t3.small, in order to increase the number of connections it can handle by doubling the memory (to 2GB).

Prod is still pinned to db.t3.medium, which has twice the memory again (4GB).

We only need to merge this if #4080 doesn't solve the scaling issue - also see that PR for further context and motivation for this change.

Also note that we'll have to manually roll out the upgrade on AWS via Pulumi CLI, since there is no CI setup for the data layer as there is for application (see the push-main workflow).

Copy link

github-actions bot commented Dec 13, 2024

Removed vultr server and associated DNS entries

Copy link
Contributor

@DafyddLlyr DafyddLlyr left a comment

Choose a reason for hiding this comment

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

Approved so that you can merge this should you wish to do so 👍

Also note that we'll have to manually roll out the upgrade on AWS via Pulumi CLI, since there is no CI setup for the data layer as there is for application (see the push-main workflow).

Very well remembered! We can run pulumi up --stack staging from the /infrastructure/data folder locally in order to achieve this - there's an intermediate step where you'll get told if there are any destructive action (the classic IaC +, -, ~).

I'd expect this to show a change ~ to instance class which would be non-destructive (i.e. not a + / -). I think that the change will be auto-applied at the next maintenance window but I'm not totally sure, the might actually be a pulumi flag to force the change.

If the absolute worst happens (very very unlikely) there's nothing irreplaceable on the staging db - it's just a clone of a subset of data from production which we can re-populate by running the "Nightly staging database sync" GitHub action.

@freemvmt
Copy link
Contributor Author

@DafyddLlyr amazing thanks for all that additional context, especially re the appropriate action to run for recovery from everything going sideways ;)

Next maintenance window isn't till next Wednesday so I probably would look to force it, so thanks on that point too!

@freemvmt
Copy link
Contributor Author

freemvmt commented Dec 13, 2024

@DafyddLlyr right you were!

image

However I apparently didn't have the right permissions to make this change from local, so I actually just made the change directly in the AWS console as we discussed the other day. Will merge this PR to reflect that the change has been applied.

@freemvmt freemvmt merged commit 8607b97 into main Dec 13, 2024
12 checks passed
@freemvmt freemvmt deleted the bump-staging-rds-instance branch December 13, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants