[terraform] Remove is-dev conditionals for DDB tables
ClosedPublic
Actions

Authored by bartek on Feb 12 2024, 8:27 AM.

Details

Reviewers

varun

Commits

rCOMMe4c8ec1585a4: [terraform] Remove is-dev conditionals for DDB tables

Summary

Resolves ENG-4545.

The identity-users-opaque2 table was dev-only and unused. Introduced in D7241 and removed in D7534
The walletAddress-index was never enabled on remote. So wallet login was working on dev only.

Test Plan

Terraform plan for staging and prod.

Diff Detail

Repository

rCOMM Comm

Branch

barthap/tf-tables

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Feb 12 2024, 8:27 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFeb 12 2024, 8:27 AM

bartek published this revision for review.Feb 12 2024, 8:28 AM

Harbormaster completed remote builds in B26738: Diff 36970.Feb 12 2024, 8:43 AM

varun accepted this revision.Feb 12 2024, 3:37 PM

This revision is now accepted and ready to land.Feb 12 2024, 3:37 PM

Closed by commit rCOMMe4c8ec1585a4: [terraform] Remove is-dev conditionals for DDB tables. · Explain WhyFeb 12 2024, 4:20 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMe4c8ec1585a4: [terraform] Remove is-dev conditionals for DDB tables.

Revision Contents
Changeset List

Path

Size

services/

terraform/

modules/

shared/

dynamodb.tf

61 lines

Diff 36970

View Options

services/terraform/modules/shared/dynamodb.tf

[terraform] Remove is-dev conditionals for DDB tablesClosedPublicActions