[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

Show All 36 Lines	function KeyserverConnectionHandler(props: Props) {
const dispatchActionPromise = useDispatchActionPromise();		const dispatchActionPromise = useDispatchActionPromise();
const callLogOut = useLogOut();		const callLogOut = useLogOut();
const keyserverAuth = useKeyserverAuth();		const keyserverAuth = useKeyserverAuth();

const hasConnectionIssue = useSelector(		const hasConnectionIssue = useSelector(
state => !!connectionSelector(keyserverID)(state)?.connectionIssue,		state => !!connectionSelector(keyserverID)(state)?.connectionIssue,
);		);
const cookie = useSelector(cookieSelector(keyserverID));		const cookie = useSelector(cookieSelector(keyserverID));
		const dataLoaded = useSelector(state => state.dataLoaded);

const keyserverDeviceToken = useSelector(deviceTokenSelector(keyserverID));		const keyserverDeviceToken = useSelector(deviceTokenSelector(keyserverID));
// We have an assumption that we should be always connected to Ashoat's		// We have an assumption that we should be always connected to Ashoat's
// keyserver. It is possible that a token which it has is correct, so we can		// keyserver. It is possible that a token which it has is correct, so we can
// try to use it. In worst case it is invalid and our push-handler will try		// try to use it. In worst case it is invalid and our push-handler will try
// to fix it.		// to fix it.
const ashoatKeyserverDeviceToken = useSelector(		const ashoatKeyserverDeviceToken = useSelector(
deviceTokenSelector(ashoatKeyserverID),		deviceTokenSelector(ashoatKeyserverID),
▲ Show 20 Lines • Show All 85 Lines • ▼ Show 20 Lines	void (async () => {
},		},
}),		}),
);		);
} catch (e) {		} catch (e) {
console.log(		console.log(
`Error while authenticating to keyserver with id ${keyserverID}`,		`Error while authenticating to keyserver with id ${keyserverID}`,
e,		e,
);		);
		if (!dataLoaded && keyserverID === ashoatKeyserverID) {
		await dispatchActionPromise(logOutActionTypes, callLogOut());
		}
} finally {		} finally {
authInProgress.current = false;		authInProgress.current = false;
}		}
})();		})();
}, [		}, [
keyserverID,		keyserverID,
identityClient,		identityClient,
olmSessionCreator,		olmSessionCreator,
cookie,		cookie,
getAuthMetadata,		getAuthMetadata,
dispatchActionPromise,		dispatchActionPromise,
keyserverAuth,		keyserverAuth,
deviceToken,		deviceToken,
calendarQuery,		calendarQuery,
isUserAuthenticated,		isUserAuthenticated,
		callLogOut,
		dataLoaded,
]);		]);

if (keyserverID !== ashoatKeyserverID) {		if (keyserverID !== ashoatKeyserverID) {
return null;		return null;
}		}
return <Socket {...rest} />;		return <Socket {...rest} />;
}		}

const Handler: React.ComponentType<Props> = React.memo<Props>(		const Handler: React.ComponentType<Props> = React.memo<Props>(
KeyserverConnectionHandler,		KeyserverConnectionHandler,
);		);

export default Handler;		export default Handler;

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