Page MenuHomePhabricator
Differential D8714

[terraform] Add S3 bucket name suffix
ClosedPublic
Actions

Authored by bartek on Aug 3 2023, 2:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 11:41 PM
Unknown Object (File)
Sun, Nov 17, 3:05 AM
Unknown Object (File)
Sun, Nov 17, 3:05 AM
Unknown Object (File)
Sun, Nov 17, 2:59 AM
Unknown Object (File)
Sat, Nov 16, 11:28 PM
Unknown Object (File)
Oct 17 2024, 1:06 AM
Unknown Object (File)
Oct 17 2024, 1:06 AM
Unknown Object (File)
Oct 17 2024, 1:02 AM
View All 66 Files
Subscribers
ashoat
tomek

Details

Reviewers
• jon
varun
Commits
rCOMMa592acaf51de: [terraform] Add S3 bucket name suffix
Summary

This stack resolves issues encountered when setting up fresh staging AWS account with Terraform.

Unfortunately, S3 bucket names are globally unique, so we can't have a separate bucket named commapp-blob in staging account.
This diff adds a configurable suffix to the bucket names and configures the suffix for staging environment only.

Details in ENG-4573.

Depends on D8713.

Test Plan

Terraform plan on all: dev, staging, prod. No changes on dev and prod.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
terraform/
modules/
shared/
2 lines
6 lines
remote/
3 lines

Diff 29664

View Options

services/terraform/modules/shared/s3.tf

Loading...
View Options

services/terraform/modules/shared/variables.tf

Loading...
View Options

services/terraform/remote/main.tf

Loading...