From 96ae1117dcbf282925c3e51916c1f13033adea30 Mon Sep 17 00:00:00 2001 From: Jamil Date: Wed, 19 Feb 2025 13:45:21 -0800 Subject: [PATCH] fix(infra): Revert to plural relays module to avoid downtime (#8201) Turns out that if we change the module structure at all, Terraform will delete all the old resources contained within it before creating new ones, because modules don't accept a `lifecycle` block. The old resources are deemed no longer "needed" and so `create_before_destroy` doesn't save us. This updates the ref to environments that contains a reverting back to the plural `module.relays[0]` structure. The way to get around this is going to have to be to duplicate the existing relays module, `terraform apply` to bring those up, and then remove the old module. That is saved for a later PR. This has been tested to achieve near-zero downtime on staging. --- terraform/environments | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/terraform/environments b/terraform/environments index 2d2d7c8c4..3f0b00c18 160000 --- a/terraform/environments +++ b/terraform/environments @@ -1 +1 @@ -Subproject commit 2d2d7c8c4471d02f20c03be998e6f257aebbabc5 +Subproject commit 3f0b00c18ebc5b0d0a403e1514b0555188607b17