[terraform] make aws rds instance class configurable for self-hosting
ClosedPublic
Actions

Authored by will on Jul 8 2024, 5:34 AM.

Details

Reviewers

bartek
varun

Commits

rCOMM58ebfd29d060: [terraform] make aws rds instance class configurable for self-hosting

Summary

This makes the rds instance size configurable. We decided that db.m6g.large was likely too large and expensive for a default setup.

Unless specified, this makes db.t4g.medium the default.

Depends on D12687

Test Plan

terraform apply. RDS instance works with keyserver as before

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

will created this revision.Jul 8 2024, 5:34 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 8 2024, 5:34 AM

will added a child revision: D12689: [terraform] create comm database on aws rds startup.Jul 8 2024, 5:38 AM

Harbormaster completed remote builds in B30192: Diff 42104.Jul 8 2024, 5:50 AM

will requested review of this revision.Jul 8 2024, 5:50 AM

varun accepted this revision.Jul 10 2024, 11:56 PM

This revision is now accepted and ready to land.Jul 10 2024, 11:56 PM

If you could remind me to make sure I set db.m6g.large when we set up my keyserver, that would be great!

In D12688#360458, @ashoat wrote:

If you could remind me to make sure I set db.m6g.large when we set up my keyserver, that would be great!

Will include as part of instructions

Closed by commit rCOMM58ebfd29d060: [terraform] make aws rds instance class configurable for self-hosting (authored by will). · Explain WhyJul 17 2024, 7:01 PM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMM58ebfd29d060: [terraform] make aws rds instance class configurable for self-hosting.

Revision Contents
Changeset List

Path

Size

services/

terraform/

self-host/

variables.tf

6 lines

Diff 42441

View Options

services/terraform/self-host/variables.tf

	fn main() {			fn main() {
	tonic_build::compile_protos("../../shared/protos/identity.proto")			tonic_build::compile_protos("../../shared/protos/identity.proto")
	.unwrap_or_else(\|e\| panic!("Failed to compile protos {:?}", e));			.unwrap_or_else(\|e\| panic!("Failed to compile identity protos {:?}", e));
				tonic_build::compile_protos("../../shared/protos/blob.proto")
				.unwrap_or_else(\|e\| panic!("Failed to compile blob protos {:?}", e));
	let _cxx_build =			let _cxx_build =
	cxx_build::bridge("src/lib.rs").flag_if_supported("-std=c++14");			cxx_build::bridge("src/lib.rs").flag_if_supported("-std=c++14");
	println!("cargo:rerun-if-changed=src/hello.c");			println!("cargo:rerun-if-changed=src/hello.c");
	}			}

[terraform] make aws rds instance class configurable for self-hostingClosedPublicActions