Details

Reviewers

bartek
varun

Commits

rCOMM0dc189b51a86: [terraform] [2/n] enable streams for DynamoDB identity-users table

Summary

Enable DynamoDB streams for the identity-users table

Depends on D9875

Test Plan

Verified on staging

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Nov 14 2023, 7:36 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 14 2023, 7:36 AM

will retitled this revision from [terraform] enable streams for DynamoDB identity-users table to [terraform] [3/n] enable streams for DynamoDB identity-users table.Nov 14 2023, 7:43 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Nov 14 2023, 7:52 AM

Harbormaster failed remote builds in B24097: Diff 33197!

will retitled this revision from [terraform] [3/n] enable streams for DynamoDB identity-users table to [terraform] [4/n] enable streams for DynamoDB identity-users table.Nov 21 2023, 2:03 AM

will retitled this revision from [terraform] [4/n] enable streams for DynamoDB identity-users table to [terraform] [3/n] enable streams for DynamoDB identity-users table.

will edited the summary of this revision. (Show Details)Nov 21 2023, 2:40 AM

will added a parent revision: D9876: [terraform] setting variables in main for opensearch.

will added a child revision: D9936: [services] [3/n] Rust opensearch indexing lambda.

will retitled this revision from [terraform] [3/n] enable streams for DynamoDB identity-users table to [terraform] [2/n] enable streams for DynamoDB identity-users table.Nov 21 2023, 8:14 AM

will edited the summary of this revision. (Show Details)

will added a parent revision: D9875: [terraform] [1/n] adding Opensearch service.

rebase

Harbormaster returned this revision to the author for changes because remote builds failed.Nov 21 2023, 8:25 PM

Harbormaster failed remote builds in B24315: Diff 33470!

Rebase

Harbormaster completed remote builds in B24318: Diff 33474.Nov 21 2023, 9:59 PM

will requested review of this revision.Nov 21 2023, 9:59 PM

bartek mentioned this in D9958: [scripts] Fix terraform lint.Nov 23 2023, 10:44 AM

I won't block you on this but could you please reformat the file before landing? 🙏

services/terraform/modules/shared/dynamodb.tf
102–105 ↗	(On Diff #33474)	This should have failed the lint but I just realized that `scripts/terraform_pre_commit.sh` is broken. I created D9958 to fix this. Can you run `terraform fmt` to reformat this?

This revision is now accepted and ready to land.Nov 23 2023, 10:47 AM

terraform fmt

Harbormaster completed remote builds in B24527: Diff 33742.Nov 27 2023, 6:45 PM

bartek mentioned this in rCOMMea2bcfa64137: [scripts] Fix terraform lint.Dec 1 2023, 8:20 AM

Whitespace

Harbormaster completed remote builds in B25158: Diff 34768.Dec 16 2023, 10:35 AM

I'm not sure this is formatted correctly. @wyilio, you may need to rebase on master in order to pick up the commit where @bartek fixed the Terraform linter precommit hook

(Apologies if I'm wrong – don't fully understand Terraform formatting guidelines)

will planned changes to this revision.Dec 18 2023, 7:29 AM

terraform fmt

This revision is now accepted and ready to land.Dec 18 2023, 8:45 PM

Harbormaster completed remote builds in B25195: Diff 34812.Dec 18 2023, 9:09 PM

Closed by commit rCOMM0dc189b51a86: [terraform] [2/n] enable streams for DynamoDB identity-users table. · Explain WhyJan 5 2024, 11:25 PM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMM0dc189b51a86: [terraform] [2/n] enable streams for DynamoDB identity-users table.

[terraform] [2/n] enable streams for DynamoDB identity-users table
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 34768

services/terraform/modules/shared/dynamodb.tf

[terraform] [2/n] enable streams for DynamoDB identity-users tableClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 34768

services/terraform/modules/shared/dynamodb.tf

[terraform] [2/n] enable streams for DynamoDB identity-users table
ClosedPublic
Actions

Revision Contents
Changeset List