Page MenuHomePhabricator
Differential D12875

[terraform] create separate lb security groups for landing and web
ClosedPublic
Actions

Authored by will on Jul 24 2024, 12:57 PM.
Tags
None
Referenced Files
F3231912: D12875.id42778.diff
Tue, Nov 12, 12:25 PM
Unknown Object (File)
Sat, Nov 9, 9:11 AM
Unknown Object (File)
Fri, Nov 8, 3:25 AM
Unknown Object (File)
Fri, Nov 1, 12:31 PM
Unknown Object (File)
Tue, Oct 22, 10:24 PM
Unknown Object (File)
Mon, Oct 14, 12:23 PM
Unknown Object (File)
Mon, Oct 14, 12:23 PM
Unknown Object (File)
Mon, Oct 14, 12:23 PM
View All 100 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMMfe8af3b57b72: [terraform] create separate lb security groups for landing and web
Summary

Originally used a shared lb-sg but this is a bad idea given that the disabling ingress approach for the migration script would disable traffic for the webapp and landing page, not just
the keyserver.

Depends on D12874

Test Plan

terraform apply and was able to access landing page and webapp at custom domains

Diff Detail

Repository
rCOMM Comm
Branch
keyserver_to_aws
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will created this revision.Jul 24 2024, 12:57 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 24 2024, 12:57 PM
Harbormaster completed remote builds in B30652: Diff 42749.Jul 24 2024, 1:15 PM
will requested review of this revision.Jul 24 2024, 1:15 PM
will added a child revision: D12876: [terraform] Making user created vpc false by default.Jul 24 2024, 1:17 PM
will added a child revision: D12877: [terraform] Update terraform.tfvars.example and use json format.Jul 24 2024, 1:19 PM
bartek accepted this revision.Jul 25 2024, 6:51 AM
bartek added inline comments.
services/terraform/self-host/landing.tf
168

Can we be more specific here?

services/terraform/self-host/webapp.tf
168

Can we be more specific here?

This revision is now accepted and ready to land.Jul 25 2024, 6:51 AM
will added inline comments.Jul 25 2024, 7:08 AM
services/terraform/self-host/landing.tf
168

Thanks for catching that. Including in next rebase

services/terraform/self-host/webapp.tf
168

Same as above. Including in next rebase

will updated this revision to Diff 42772.Jul 25 2024, 7:12 AM

update descriptions for landing and webapp security groups

Harbormaster failed remote builds in B30668: Diff 42772!Jul 25 2024, 7:13 AM
will updated this revision to Diff 42777.Jul 25 2024, 9:37 AM

rebase

Harbormaster failed remote builds in B30673: Diff 42777!Jul 25 2024, 9:39 AM
will updated this revision to Diff 42784.Jul 25 2024, 11:15 AM

rebase

Harbormaster completed remote builds in B30678: Diff 42784.Jul 25 2024, 11:32 AM
Closed by commit rCOMMfe8af3b57b72: [terraform] create separate lb security groups for landing and web (authored by will). · Explain WhyJul 25 2024, 2:22 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMMfe8af3b57b72: [terraform] create separate lb security groups for landing and web.

Revision Contents
Changeset List

PathSize
services/
terraform/
self-host/
1 line
22 lines
22 lines

Diff 42749

View Options

services/terraform/self-host/aws_lb.tf

Loading...
View Options

services/terraform/self-host/landing.tf

Loading...
View Options

services/terraform/self-host/webapp.tf

Loading...