Differential D9216

[terraform] Deploy Tunnelbroker DynamoDB table
ClosedPublic
Actions

Authored by bartek on Sep 15 2023, 2:36 AM.

Tags

None

Referenced Files

	F3599090: D9216.id31165.diff
	Tue, Dec 31, 2:07 AM

	F3599088: D9216.id31162.diff
	Tue, Dec 31, 2:07 AM

	F3599045: D9216.id.diff
	Tue, Dec 31, 2:07 AM

	F3598992: D9216.diff
	Tue, Dec 31, 1:59 AM

	Unknown Object (File)
	Dec 1 2024, 9:33 AM

	Unknown Object (File)
	Nov 29 2024, 2:50 AM

	Unknown Object (File)
	Nov 29 2024, 12:05 AM

	Unknown Object (File)
	Nov 15 2024, 1:58 AM

View All 60 Files

Subscribers

Details

Reviewers

kamil
• jon

Commits

rCOMM7e0a7cce56ff: [terraform] Deploy Tunnelbroker DynamoDB table

Summary

It turns out that this table was missing on AWS. Interestingly, @kamil noticed that tunnelbroker actually hasn't been saving anything to dynamodb and the test is broken. This is why I didn't notice that the table was missing - tunnelbroker wasn't failing without the table.

Test Plan

Terraform plan + terraform apply on staging created the table.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Sep 15 2023, 2:36 AM

bartek held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptSep 15 2023, 2:36 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

bartek published this revision for review.Sep 15 2023, 2:37 AM

kamil accepted this revision.Sep 15 2023, 2:47 AM

This revision is now accepted and ready to land.Sep 15 2023, 2:47 AM

Harbormaster completed remote builds in B22651: Diff 31162.Sep 15 2023, 2:54 AM

Closed by commit rCOMM7e0a7cce56ff: [terraform] Deploy Tunnelbroker DynamoDB table (authored by bartek). · Explain WhySep 15 2023, 3:45 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM7e0a7cce56ff: [terraform] Deploy Tunnelbroker DynamoDB table.

Revision Contents
Changeset List

Path

Size

services/

terraform/

modules/

shared/

3 lines

1 line

Diff 31165

services/terraform/modules/shared/dynamodb.tf

Loading...

services/terraform/modules/shared/outputs.tf

Loading...