Run webapp on the comm services cluster. Will not land until web.comm.app has been properly set up on prod
This will require a new keyserver image push with https://phab.comm.dev/D12927
Depends on D12906
Paths
| Differential D12928 Authored by will on Jul 29 2024, 8:43 PM.
Tags None Referenced Files
Details Summary Run webapp on the comm services cluster. Will not land until web.comm.app has been properly set up on prod This will require a new keyserver image push with https://phab.comm.dev/D12927 Depends on D12906 Test Plan terraform apply. Switched comm.software to point to web app load balancer and accessed the working web app. Used wyilio/keysever:1.0 for testing
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptJul 29 2024, 8:43 PM2024-07-29 20:43:49 (UTC-7) will edited the summary of this revision. (Show Details)Jul 29 2024, 8:44 PM2024-07-29 20:44:14 (UTC-7) will added a parent revision: D12927: [keyserver] Do not verify identity logged for webapp and landing.
will added a child revision: D12929: [terraform] use module approach for landing.Jul 29 2024, 8:50 PM2024-07-29 20:50:03 (UTC-7) Harbormaster completed remote builds in B30777: Diff 42920.Jul 29 2024, 9:10 PM2024-07-29 21:10:48 (UTC-7) will edited the summary of this revision. (Show Details)Jul 29 2024, 9:12 PM2024-07-29 21:12:45 (UTC-7) Harbormaster completed remote builds in B30779: Diff 42922.Jul 29 2024, 9:13 PM2024-07-29 21:13:17 (UTC-7) will edited the summary of this revision. (Show Details)Jul 29 2024, 9:15 PM2024-07-29 21:15:39 (UTC-7) Comment Actions General question: webapp_environment_vars = merge(local.secrets.webapp_secrets, { ... }) ? I'm not saying your approach is bad, I just want to understand the motivation
Comment Actions
Wasn't something I exactly considered. I think my main motivation was keeping it as a .env format, but let me know if you think going with the secrets.json approach would be better. It would simplify the process of the null resource creating a .env file every time.
Harbormaster failed remote builds in B30858: Diff 43025!Aug 1 2024, 11:32 AM2024-08-01 11:32:50 (UTC-7) Harbormaster failed remote builds in B30860: Diff 43027!Aug 1 2024, 12:23 PM2024-08-01 12:23:19 (UTC-7)
This revision is now accepted and ready to land.Aug 2 2024, 10:35 AM2024-08-02 10:35:41 (UTC-7) will edited the summary of this revision. (Show Details)Aug 5 2024, 11:57 AM2024-08-05 11:57:21 (UTC-7) will added a parent revision: D12906: [terraform] Create module for webapp/landing and convert webapp to module. will removed a child revision: D12930: [terraform] deploy landing on comm services cluster.Aug 5 2024, 12:00 PM2024-08-05 12:00:06 (UTC-7)
Harbormaster completed remote builds in B30968: Diff 43197.Aug 6 2024, 7:00 PM2024-08-06 19:00:51 (UTC-7) Harbormaster completed remote builds in B30969: Diff 43198.Aug 6 2024, 7:03 PM2024-08-06 19:03:39 (UTC-7) Closed by commit rCOMM42499907ef11: [keyserver] Run webapp through comm services cluster (authored by will). · Explain WhyAug 6 2024, 7:43 PM2024-08-06 19:43:06 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 43198 services/terraform/remote/aws_iam.tf
services/terraform/remote/secrets.json
services/terraform/remote/service_webapp.tf
|