[Tunnelbroker] Remove existing tunnelbroker tables
ClosedPublic
Actions

Authored by • jon on May 4 2023, 9:50 AM.

Details

Reviewers

bartek
varun

Commits

rCOMM0ca7602266d2: [Tunnelbroker] Remove existing tunnelbroker tables

Summary

Remove stale tables from tf configuration. To avoid
data collision with existing data, use new naming.

Part of: https://linear.app/comm/issue/ENG-3823

Test Plan

# verify that terraform works
cd services/terraftorm
./run.sh

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jon created this revision.May 4 2023, 9:50 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMay 4 2023, 9:50 AM

Harbormaster returned this revision to the author for changes because remote builds failed.May 4 2023, 10:06 AM

Harbormaster failed remote builds in B19047: Diff 26079!

• jon added a parent revision: D7734: [Tunnelbroker] Add initial dynamodb client.May 5 2023, 1:36 PM

• jon added a child revision: D7718: [Tunnelbroker] Create undelivered messages table.

Rebase to history without failing JS commmits

Harbormaster completed remote builds in B19083: Diff 26131.May 5 2023, 1:58 PM

• jon requested review of this revision.May 5 2023, 1:58 PM

bartek accepted this revision.May 8 2023, 7:20 AM

This revision is now accepted and ready to land.May 8 2023, 7:20 AM

Rebase on master

Harbormaster completed remote builds in B19313: Diff 26427.May 12 2023, 4:50 PM

Closed by commit rCOMM0ca7602266d2: [Tunnelbroker] Remove existing tunnelbroker tables (authored by • jon). · Explain WhyMay 24 2023, 2:36 PM

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMM0ca7602266d2: [Tunnelbroker] Remove existing tunnelbroker tables.

Revision Contents
Changeset List

Path

Size

services/

terraform/

dynamodb-test.tf

69 lines

dynamodb.tf

69 lines

Diff 27020

View Options

services/terraform/dynamodb-test.tf