Paths

Table of Contentst

Differential D11028

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

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

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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/

61 lines

Diff 37005

services/terraform/modules/shared/dynamodb.tf

Loading...