feedback. remove unnecessary homebrew install deps script

review feedback

rebase

only create vpc, subnets, and internet gateway if user created option is true

feedback

reduce line count

include logic to disable traffic until all nodes available

create secondary resources after primary

typo

include security group

No longer needed. Included in prior diff

Add option for user created vpc to avoid migration of prod aws rds instance to new vpc

Looks good!

Just saw https://linear.app/comm/issue/ENG-8810/unable-to-login#comment-47abbe95. Unrelated to the actual code in this diff but we should probably make a fix a dependency for this one

run this command every time

Directly synced with Varun. Looks good. Before landing make sure to remove need for unnecessary partition key check and add a comment for changes starting from line 340 in services/identity/src/database.rs

set to false initially

rebase

rebase with new docker image push

revamp. No longer reenabling default features. Instead just adding to rustls-tls to identity Cargo.toml

recompile backup_client from workspace cargo.toml change

add email subscription

fix filters and log group names

change pattern to "dns error"

We actually do want this as "dns error"

In D12563#355851, @bartek wrote:

Are you planning to use moved { } block? Or use terraform state mv to move it without recreating?

In D12457#353784, @ashoat wrote:

Why not the latest version?

This makes a lot of sense. Not sure why I didn't consider this on first implementation but glad you caught it now.