Page MenuHomePhabricator
Differential D9066

[backup] Setup terraform
ClosedPublic
Actions

Authored by michal on Sep 1 2023, 7:20 AM.
Tags
None
Referenced Files
F3521861: D9066.diff
Mon, Dec 23, 5:10 AM
Unknown Object (File)
Sat, Dec 14, 11:56 PM
Unknown Object (File)
Sat, Dec 14, 11:56 PM
Unknown Object (File)
Sat, Dec 14, 11:56 PM
Unknown Object (File)
Sat, Dec 14, 11:56 PM
Unknown Object (File)
Sat, Dec 14, 11:55 PM
Unknown Object (File)
Sat, Dec 14, 11:39 PM
Unknown Object (File)
Mon, Dec 9, 11:33 PM
View All 99 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
• patryk
kamil
• jon
Commits
rCOMMb06aa5a01d4b: [backup] Setup terraform
Summary

ENG-4513

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
rCOMM Comm
Branch
michal/backup
Lint
No Lint Coverage
Unit
No Test Coverage

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
michal updated this revision to Diff 30704.Sep 1 2023, 7:22 AM

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
bartek accepted this revision.Sep 4 2023, 4:38 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.
michal updated this revision to Diff 30829.Sep 7 2023, 2:30 AM

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

PathSize
services/
terraform/
modules/
shared/
3 lines
remote/
28 lines
174 lines

Diff 30829

View Options

services/terraform/modules/shared/outputs.tf

Loading...
View Options

services/terraform/remote/aws_iam.tf

Loading...
View Options

services/terraform/remote/service_backup.tf

Loading...