[terraform] Add Backup to ECS Service Connect
ClosedPublic
Actions

Authored by bartek on Aug 5 2024, 4:56 AM.

Details

Reviewers

will
varun
kamil

Commits

rCOMM61dc78e5062b: [terraform] Add Backup to ECS Service Connect

Summary

Needed to make Backup service discoverable in ECS Service Connect, to let Identity reach it when deleting user data.

Depends on D12972

Test Plan

terraform plan + tested on staging

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Aug 5 2024, 4:56 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 5 2024, 4:56 AM

bartek published this revision for review.Aug 5 2024, 5:09 AM

Looks good but I am not experienced enough with terraform - leaving this to @will's expertise

Looks good to me!

This revision is now accepted and ready to land.Aug 5 2024, 6:19 AM

This revision was landed with ongoing or failed builds.Aug 5 2024, 6:28 AM

Closed by commit rCOMM61dc78e5062b: [terraform] Add Backup to ECS Service Connect (authored by bartek). · Explain Why

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM61dc78e5062b: [terraform] Add Backup to ECS Service Connect.

Harbormaster failed remote builds in B30911: Diff 43091!Aug 5 2024, 6:45 AM

Revision Contents
Changeset List

Path

Size

services/

terraform/

remote/

service_backup.tf

25 lines

service_identity.tf

4 lines

Diff 43127

View Options

services/terraform/remote/service_backup.tf