Differential D9066

[backup] Setup terraform
ClosedPublic
Actions

Authored by michal on Sep 1 2023, 7:20 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 6, 4:27 AM

	Unknown Object (File)
	Wed, Nov 6, 4:27 AM

	Unknown Object (File)
	Wed, Nov 6, 4:27 AM

	Unknown Object (File)
	Wed, Nov 6, 4:27 AM

	Unknown Object (File)
	Wed, Nov 6, 4:26 AM

	Unknown Object (File)
	Wed, Nov 6, 4:22 AM

	Unknown Object (File)
	Oct 1 2024, 1:35 PM

	Unknown Object (File)
	Oct 1 2024, 8:35 AM

View All 87 Files

Subscribers

Details

Reviewers

bartek
• patryk
kamil
• jon

Commits

rCOMMb06aa5a01d4b: [backup] Setup terraform

Summary

Setup terraform for backup service. Mostly copy paste from D9040, with report-specific things removed.

Depends on D9065

Test Plan

Run on staging, make a few test calls to check if it works. Check with the dev version of the app if the backups work.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

michal created this revision.Sep 1 2023, 7:20 AM

Herald added a reviewer: • jon. · View Herald TranscriptSep 1 2023, 7:20 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

Add new lines

michal added inline comments.Sep 1 2023, 7:23 AM

services/terraform/remote/aws_iam.tf
151 ↗	(On Diff #30703)	Required for queries on indexes of backup table SO answer
services/terraform/remote/service_backup.tf
4 ↗	(On Diff #30703)	Temporary until @ashoat updates permissions for the backup repository on docker hub.

Harbormaster completed remote builds in B22349: Diff 30703.Sep 1 2023, 7:37 AM

Harbormaster completed remote builds in B22350: Diff 30704.

michal requested review of this revision.Sep 1 2023, 7:39 AM

kamil added a child revision: D9075: [client-backup] set backup-service prod URL.Sep 4 2023, 3:43 AM

LGTM

This revision is now accepted and ready to land.Sep 4 2023, 4:38 AM

michal added a parent revision: D9038: [terraform] Set up ECS Service Connect.Sep 7 2023, 2:09 AM

michal added a parent revision: D9063: [backup] Update backup dockerfile.

Change temporary docker repo to commapp

Harbormaster completed remote builds in B22422: Diff 30829.Sep 7 2023, 2:48 AM

Closed by commit rCOMMb06aa5a01d4b: [backup] Setup terraform (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhySep 7 2023, 5:03 AM

This revision was automatically updated to reflect the committed changes.

Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMMb06aa5a01d4b: [backup] Setup terraform.

Revision Contents
Changeset List

Path

Size

services/

terraform/

modules/

shared/

3 lines

remote/

28 lines

service_backup.tf

174 lines

Diff 30831

services/terraform/modules/shared/outputs.tf

Loading...

services/terraform/remote/aws_iam.tf

Loading...

services/terraform/remote/service_backup.tf

Loading...