Merge pull request #2700 from wellcomecollection/Remove-DLQ-alarms-fr…

…om-pipeline-services Remove DLQ alarms from pipeline services
wellcomecollection · Sep 9, 2024 · 6bbc368 · 6bbc368
2 parents 6b82534 + acd9a46
commit 6bbc368
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/infrastructure/modules/scaling_service/queue.tf b/infrastructure/modules/scaling_service/queue.tf
@@ -1,5 +1,5 @@
 module "input_queue" {
-  source = "github.com/wellcomecollection/terraform-aws-sqs//queue?ref=v1.2.1"
+  source = "github.com/wellcomecollection/terraform-aws-sqs//queue?ref=v1.4.0"
 
   queue_name = var.queue_config.name
 

diff --git a/pipeline/terraform/modules/services_with_manager/variables.tf b/pipeline/terraform/modules/services_with_manager/variables.tf
@@ -155,5 +155,6 @@ variable "max_receive_count" {
 }
 
 variable "dlq_alarm_topic_arn" {
-  type = string
+  type    = string
+  default = null
 }
diff --git a/pipeline/terraform/modules/stack/locals.tf b/pipeline/terraform/modules/stack/locals.tf
@@ -177,7 +177,7 @@ locals {
   monitoring_config = {
     shared_logging_secrets = local.shared_infra["shared_secrets_logging"]
     logging_cluster_id     = local.shared_infra["logging_cluster_id"]
-    dlq_alarm_arn          = data.terraform_remote_state.monitoring.outputs.platform_dlq_alarm_topic_arn
+    dlq_alarm_arn          = null
   }
 
   network_config = {