From fe3a7cae89714793f33408a669e6fafe96b3ae8d Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Fri, 8 May 2026 13:53:43 +0100 Subject: [PATCH 1/7] CCM-14783:SQS Module Update to Include Alarms --- .../terraform/components/api/module_sqs_amendments.tf | 6 ++++-- .../components/api/module_sqs_letter_status_updates.tf | 6 ++++-- .../terraform/components/api/module_sqs_letter_updates.tf | 6 ++++-- .../components/api/module_sqs_supplier_allocator.tf | 6 ++++-- 4 files changed, 16 insertions(+), 8 deletions(-) diff --git a/infrastructure/terraform/components/api/module_sqs_amendments.tf b/infrastructure/terraform/components/api/module_sqs_amendments.tf index 8fca10e9d..d50d430c8 100644 --- a/infrastructure/terraform/components/api/module_sqs_amendments.tf +++ b/infrastructure/terraform/components/api/module_sqs_amendments.tf @@ -1,6 +1,6 @@ # Queue to transport letter status amendment messages module "amendments_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" name = "amendments_queue" @@ -12,5 +12,7 @@ module "amendments_queue" { sqs_kms_key_arn = module.kms.key_arn - create_dlq = true + create_dlq = true + enable_dlq_alarm = false + enable_queue_oldest_message_alarm = false } diff --git a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf index db7d27a85..bcc2f7cce 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf @@ -1,7 +1,7 @@ # Queue to transport update letter status messages. Now replaced by module.amendments_queue. # This queue will not be removed just yet, to allow it to be drained following the release in which module.amendments_queue replaces it. module "letter_status_updates_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" name = "letter_status_updates_queue" @@ -13,5 +13,7 @@ module "letter_status_updates_queue" { sqs_kms_key_arn = module.kms.key_arn - create_dlq = true + create_dlq = true + enable_dlq_alarm = false + enable_queue_oldest_message_alarm = false } diff --git a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf index 1e886d61e..9ebd5667f 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf @@ -1,5 +1,5 @@ module "sqs_letter_updates" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component @@ -12,7 +12,9 @@ module "sqs_letter_updates" { visibility_timeout_seconds = 60 - create_dlq = true + create_dlq = true + enable_dlq_alarm = false + enable_queue_oldest_message_alarm = false sqs_policy_overload = data.aws_iam_policy_document.letter_updates_queue_policy.json } diff --git a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf index 565038d7b..edf0f15d8 100644 --- a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf +++ b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf @@ -1,5 +1,5 @@ module "sqs_supplier_allocator" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component @@ -12,7 +12,9 @@ module "sqs_supplier_allocator" { visibility_timeout_seconds = 60 - create_dlq = true + create_dlq = true + enable_dlq_alarm = false + enable_queue_oldest_message_alarm = false sqs_policy_overload = data.aws_iam_policy_document.supplier_allocator_queue_policy.json } From bf84566d3cba3530adbe2ef2727b9e5fe7653889 Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Fri, 8 May 2026 13:53:55 +0100 Subject: [PATCH 2/7] CCM-14783:SQS Module Update to Include Alarms --- infrastructure/terraform/components/api/README.md | 8 ++++---- .../terraform/components/api/module_sqs_letter_updates.tf | 2 +- .../components/api/module_sqs_supplier_allocator.tf | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/infrastructure/terraform/components/api/README.md b/infrastructure/terraform/components/api/README.md index 00196a0bc..f1f031e76 100644 --- a/infrastructure/terraform/components/api/README.md +++ b/infrastructure/terraform/components/api/README.md @@ -52,7 +52,7 @@ No requirements. | Name | Source | Version | |------|--------|---------| | [amendment\_event\_transformer](#module\_amendment\_event\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | -| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip | n/a | +| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | | [authorizer\_lambda](#module\_authorizer\_lambda) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [ddb\_alarms\_idempotency](#module\_ddb\_alarms\_idempotency) | ../../modules/alarms-ddb | n/a | | [ddb\_alarms\_letter\_queue](#module\_ddb\_alarms\_letter\_queue) | ../../modules/alarms-ddb | n/a | @@ -68,7 +68,7 @@ No requirements. | [get\_status](#module\_get\_status) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [kms](#module\_kms) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-kms.zip | n/a | | [lambda\_alarms](#module\_lambda\_alarms) | ../../modules/alarms-lambda | n/a | -| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip | n/a | +| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | | [letter\_updates\_transformer](#module\_letter\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [mi\_updates\_transformer](#module\_mi\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-lambda.zip | n/a | | [patch\_letter](#module\_patch\_letter) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | @@ -76,8 +76,8 @@ No requirements. | [post\_mi](#module\_post\_mi) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [s3bucket\_test\_letters](#module\_s3bucket\_test\_letters) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-s3bucket.zip | n/a | | [sqs\_alarms](#module\_sqs\_alarms) | ../../modules/alarms-sqs | n/a | -| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip | n/a | -| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.0.6/terraform-sqs.zip | n/a | +| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | +| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | | [supplier\_allocator](#module\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [supplier\_ssl](#module\_supplier\_ssl) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-ssl.zip | n/a | | [update\_letter\_queue](#module\_update\_letter\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | diff --git a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf index 9ebd5667f..a51c8b53f 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf @@ -15,7 +15,7 @@ module "sqs_letter_updates" { create_dlq = true enable_dlq_alarm = false enable_queue_oldest_message_alarm = false - sqs_policy_overload = data.aws_iam_policy_document.letter_updates_queue_policy.json + sqs_policy_overload = data.aws_iam_policy_document.letter_updates_queue_policy.json } data "aws_iam_policy_document" "letter_updates_queue_policy" { diff --git a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf index edf0f15d8..47d445fdb 100644 --- a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf +++ b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf @@ -15,7 +15,7 @@ module "sqs_supplier_allocator" { create_dlq = true enable_dlq_alarm = false enable_queue_oldest_message_alarm = false - sqs_policy_overload = data.aws_iam_policy_document.supplier_allocator_queue_policy.json + sqs_policy_overload = data.aws_iam_policy_document.supplier_allocator_queue_policy.json } data "aws_iam_policy_document" "supplier_allocator_queue_policy" { From 4e3da2c74b4d53a80dff470f9022649de2899cb7 Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Fri, 8 May 2026 14:05:54 +0100 Subject: [PATCH 3/7] CCM-14783:SQS Module Update to Include Alarms 3.1.4 --- infrastructure/terraform/components/api/README.md | 8 ++++---- .../terraform/components/api/module_sqs_amendments.tf | 2 +- .../components/api/module_sqs_letter_status_updates.tf | 2 +- .../terraform/components/api/module_sqs_letter_updates.tf | 2 +- .../components/api/module_sqs_supplier_allocator.tf | 2 +- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/infrastructure/terraform/components/api/README.md b/infrastructure/terraform/components/api/README.md index f1f031e76..404b51be8 100644 --- a/infrastructure/terraform/components/api/README.md +++ b/infrastructure/terraform/components/api/README.md @@ -52,7 +52,7 @@ No requirements. | Name | Source | Version | |------|--------|---------| | [amendment\_event\_transformer](#module\_amendment\_event\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | -| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | +| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | | [authorizer\_lambda](#module\_authorizer\_lambda) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [ddb\_alarms\_idempotency](#module\_ddb\_alarms\_idempotency) | ../../modules/alarms-ddb | n/a | | [ddb\_alarms\_letter\_queue](#module\_ddb\_alarms\_letter\_queue) | ../../modules/alarms-ddb | n/a | @@ -68,7 +68,7 @@ No requirements. | [get\_status](#module\_get\_status) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [kms](#module\_kms) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-kms.zip | n/a | | [lambda\_alarms](#module\_lambda\_alarms) | ../../modules/alarms-lambda | n/a | -| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | +| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | | [letter\_updates\_transformer](#module\_letter\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [mi\_updates\_transformer](#module\_mi\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-lambda.zip | n/a | | [patch\_letter](#module\_patch\_letter) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | @@ -76,8 +76,8 @@ No requirements. | [post\_mi](#module\_post\_mi) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [s3bucket\_test\_letters](#module\_s3bucket\_test\_letters) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-s3bucket.zip | n/a | | [sqs\_alarms](#module\_sqs\_alarms) | ../../modules/alarms-sqs | n/a | -| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | -| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip | n/a | +| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | +| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | | [supplier\_allocator](#module\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [supplier\_ssl](#module\_supplier\_ssl) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-ssl.zip | n/a | | [update\_letter\_queue](#module\_update\_letter\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | diff --git a/infrastructure/terraform/components/api/module_sqs_amendments.tf b/infrastructure/terraform/components/api/module_sqs_amendments.tf index d50d430c8..c3eb2a69b 100644 --- a/infrastructure/terraform/components/api/module_sqs_amendments.tf +++ b/infrastructure/terraform/components/api/module_sqs_amendments.tf @@ -1,6 +1,6 @@ # Queue to transport letter status amendment messages module "amendments_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" name = "amendments_queue" diff --git a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf index bcc2f7cce..da3f015ae 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf @@ -1,7 +1,7 @@ # Queue to transport update letter status messages. Now replaced by module.amendments_queue. # This queue will not be removed just yet, to allow it to be drained following the release in which module.amendments_queue replaces it. module "letter_status_updates_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" name = "letter_status_updates_queue" diff --git a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf index a51c8b53f..287de432b 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf @@ -1,5 +1,5 @@ module "sqs_letter_updates" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component diff --git a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf index 47d445fdb..48a58f2df 100644 --- a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf +++ b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf @@ -1,5 +1,5 @@ module "sqs_supplier_allocator" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.3/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component From 972d84eed1f0e608c703cab906ad7ecc2cc5a0d6 Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Mon, 11 May 2026 10:46:33 +0100 Subject: [PATCH 4/7] CCM-14783:SQS Module Update to Include Alarms 3.1.4 --- .../components/api/module_sqs_alarms.tf | 1 - .../components/api/module_sqs_amendments.tf | 1 - .../api/module_sqs_letter_status_updates.tf | 1 - .../api/module_sqs_letter_updates.tf | 1 - .../api/module_sqs_supplier_allocator.tf | 1 - .../terraform/modules/alarms-sqs/README.md | 2 -- .../cloudwatch_metric_alarm_dlq_depth.tf | 23 ------------------- .../terraform/modules/alarms-sqs/variables.tf | 10 -------- 8 files changed, 40 deletions(-) delete mode 100644 infrastructure/terraform/modules/alarms-sqs/cloudwatch_metric_alarm_dlq_depth.tf diff --git a/infrastructure/terraform/components/api/module_sqs_alarms.tf b/infrastructure/terraform/components/api/module_sqs_alarms.tf index 28b948640..d0cda0ed3 100644 --- a/infrastructure/terraform/components/api/module_sqs_alarms.tf +++ b/infrastructure/terraform/components/api/module_sqs_alarms.tf @@ -4,6 +4,5 @@ module "sqs_alarms" { alarm_prefix = local.csi queue_name = each.value - dlq_queue_name = replace(each.value, "-queue", "-dlq") tags = local.default_tags } diff --git a/infrastructure/terraform/components/api/module_sqs_amendments.tf b/infrastructure/terraform/components/api/module_sqs_amendments.tf index c3eb2a69b..3a7d866f2 100644 --- a/infrastructure/terraform/components/api/module_sqs_amendments.tf +++ b/infrastructure/terraform/components/api/module_sqs_amendments.tf @@ -13,6 +13,5 @@ module "amendments_queue" { sqs_kms_key_arn = module.kms.key_arn create_dlq = true - enable_dlq_alarm = false enable_queue_oldest_message_alarm = false } diff --git a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf index da3f015ae..0318c46a8 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf @@ -14,6 +14,5 @@ module "letter_status_updates_queue" { sqs_kms_key_arn = module.kms.key_arn create_dlq = true - enable_dlq_alarm = false enable_queue_oldest_message_alarm = false } diff --git a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf index 287de432b..b3077a083 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf @@ -13,7 +13,6 @@ module "sqs_letter_updates" { visibility_timeout_seconds = 60 create_dlq = true - enable_dlq_alarm = false enable_queue_oldest_message_alarm = false sqs_policy_overload = data.aws_iam_policy_document.letter_updates_queue_policy.json } diff --git a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf index 48a58f2df..d0982c14a 100644 --- a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf +++ b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf @@ -13,7 +13,6 @@ module "sqs_supplier_allocator" { visibility_timeout_seconds = 60 create_dlq = true - enable_dlq_alarm = false enable_queue_oldest_message_alarm = false sqs_policy_overload = data.aws_iam_policy_document.supplier_allocator_queue_policy.json } diff --git a/infrastructure/terraform/modules/alarms-sqs/README.md b/infrastructure/terraform/modules/alarms-sqs/README.md index 6ddbea4ca..2c721eadb 100644 --- a/infrastructure/terraform/modules/alarms-sqs/README.md +++ b/infrastructure/terraform/modules/alarms-sqs/README.md @@ -16,8 +16,6 @@ | [age\_anomaly\_sensitivity](#input\_age\_anomaly\_sensitivity) | n/a | `number` | `3` | no | | [age\_period\_seconds](#input\_age\_period\_seconds) | n/a | `number` | `900` | no | | [alarm\_prefix](#input\_alarm\_prefix) | n/a | `string` | n/a | yes | -| [dlq\_queue\_name](#input\_dlq\_queue\_name) | n/a | `string` | `null` | no | -| [dlq\_visible\_threshold](#input\_dlq\_visible\_threshold) | n/a | `number` | `0` | no | | [queue\_name](#input\_queue\_name) | n/a | `string` | n/a | yes | | [tags](#input\_tags) | n/a | `map(string)` | `{}` | no | ## Modules diff --git a/infrastructure/terraform/modules/alarms-sqs/cloudwatch_metric_alarm_dlq_depth.tf b/infrastructure/terraform/modules/alarms-sqs/cloudwatch_metric_alarm_dlq_depth.tf deleted file mode 100644 index 41b1251ba..000000000 --- a/infrastructure/terraform/modules/alarms-sqs/cloudwatch_metric_alarm_dlq_depth.tf +++ /dev/null @@ -1,23 +0,0 @@ -resource "aws_cloudwatch_metric_alarm" "dlq_depth" { - count = var.dlq_queue_name == null ? 0 : 1 - alarm_name = "${var.alarm_prefix}-sqs-${var.dlq_queue_name}-dlq-depth" - alarm_description = "RELIABILITY: SQS DLQ has messages" - - namespace = "AWS/SQS" - metric_name = "ApproximateNumberOfMessagesVisible" - statistic = "Sum" - period = 60 - - evaluation_periods = 1 - threshold = var.dlq_visible_threshold - comparison_operator = "GreaterThanThreshold" - treat_missing_data = "notBreaching" - - dimensions = { QueueName = var.dlq_queue_name } - - actions_enabled = false - alarm_actions = [] - ok_actions = [] - insufficient_data_actions = [] - tags = var.tags -} diff --git a/infrastructure/terraform/modules/alarms-sqs/variables.tf b/infrastructure/terraform/modules/alarms-sqs/variables.tf index 364fc6153..c635c3973 100644 --- a/infrastructure/terraform/modules/alarms-sqs/variables.tf +++ b/infrastructure/terraform/modules/alarms-sqs/variables.tf @@ -6,11 +6,6 @@ variable "queue_name" { type = string } -variable "dlq_queue_name" { - type = string - default = null -} - variable "tags" { type = map(string) default = {} @@ -35,8 +30,3 @@ variable "age_anomaly_datapoints_to_alarm" { type = number default = 3 } - -variable "dlq_visible_threshold" { - type = number - default = 0 -} From d17e1533657f149a650a7f6e7a6497836ac6462b Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Mon, 11 May 2026 10:46:43 +0100 Subject: [PATCH 5/7] CCM-14783:SQS Module Update to Include Alarms 3.1.4 --- .../terraform/components/api/module_sqs_alarms.tf | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/infrastructure/terraform/components/api/module_sqs_alarms.tf b/infrastructure/terraform/components/api/module_sqs_alarms.tf index d0cda0ed3..e9da40596 100644 --- a/infrastructure/terraform/components/api/module_sqs_alarms.tf +++ b/infrastructure/terraform/components/api/module_sqs_alarms.tf @@ -2,7 +2,7 @@ module "sqs_alarms" { for_each = local.alarms_enabled ? local.sqs_alarm_targets : {} source = "../../modules/alarms-sqs" - alarm_prefix = local.csi - queue_name = each.value - tags = local.default_tags + alarm_prefix = local.csi + queue_name = each.value + tags = local.default_tags } From 374b347e71ca89b8167c7ec59e405e61d763f17d Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Mon, 11 May 2026 11:19:04 +0100 Subject: [PATCH 6/7] CCM-14783:SQS Module Update Use New Outputs --- .../terraform/components/api/module_sqs_amendments.tf | 2 +- .../components/api/module_sqs_letter_status_updates.tf | 2 +- .../terraform/components/api/module_sqs_letter_updates.tf | 2 +- .../terraform/components/api/module_sqs_supplier_allocator.tf | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/infrastructure/terraform/components/api/module_sqs_amendments.tf b/infrastructure/terraform/components/api/module_sqs_amendments.tf index 3a7d866f2..8e30bb0d9 100644 --- a/infrastructure/terraform/components/api/module_sqs_amendments.tf +++ b/infrastructure/terraform/components/api/module_sqs_amendments.tf @@ -1,6 +1,6 @@ # Queue to transport letter status amendment messages module "amendments_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip" name = "amendments_queue" diff --git a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf index 0318c46a8..59c0cfef0 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_status_updates.tf @@ -1,7 +1,7 @@ # Queue to transport update letter status messages. Now replaced by module.amendments_queue. # This queue will not be removed just yet, to allow it to be drained following the release in which module.amendments_queue replaces it. module "letter_status_updates_queue" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip" name = "letter_status_updates_queue" diff --git a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf index b3077a083..8e94af340 100644 --- a/infrastructure/terraform/components/api/module_sqs_letter_updates.tf +++ b/infrastructure/terraform/components/api/module_sqs_letter_updates.tf @@ -1,5 +1,5 @@ module "sqs_letter_updates" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component diff --git a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf index d0982c14a..653417a27 100644 --- a/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf +++ b/infrastructure/terraform/components/api/module_sqs_supplier_allocator.tf @@ -1,5 +1,5 @@ module "sqs_supplier_allocator" { - source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip" + source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip" aws_account_id = var.aws_account_id component = var.component From 270db8d131681838c16bcca7e51791acf4c7a652 Mon Sep 17 00:00:00 2001 From: jamesthompson26-nhs Date: Mon, 11 May 2026 11:19:16 +0100 Subject: [PATCH 7/7] CCM-14783:SQS Module Update Use New Outputs --- infrastructure/terraform/components/api/README.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/infrastructure/terraform/components/api/README.md b/infrastructure/terraform/components/api/README.md index 404b51be8..8071127c5 100644 --- a/infrastructure/terraform/components/api/README.md +++ b/infrastructure/terraform/components/api/README.md @@ -52,7 +52,7 @@ No requirements. | Name | Source | Version | |------|--------|---------| | [amendment\_event\_transformer](#module\_amendment\_event\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | -| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | +| [amendments\_queue](#module\_amendments\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip | n/a | | [authorizer\_lambda](#module\_authorizer\_lambda) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [ddb\_alarms\_idempotency](#module\_ddb\_alarms\_idempotency) | ../../modules/alarms-ddb | n/a | | [ddb\_alarms\_letter\_queue](#module\_ddb\_alarms\_letter\_queue) | ../../modules/alarms-ddb | n/a | @@ -68,7 +68,7 @@ No requirements. | [get\_status](#module\_get\_status) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [kms](#module\_kms) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-kms.zip | n/a | | [lambda\_alarms](#module\_lambda\_alarms) | ../../modules/alarms-lambda | n/a | -| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | +| [letter\_status\_updates\_queue](#module\_letter\_status\_updates\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip | n/a | | [letter\_updates\_transformer](#module\_letter\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [mi\_updates\_transformer](#module\_mi\_updates\_transformer) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-lambda.zip | n/a | | [patch\_letter](#module\_patch\_letter) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | @@ -76,8 +76,8 @@ No requirements. | [post\_mi](#module\_post\_mi) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [s3bucket\_test\_letters](#module\_s3bucket\_test\_letters) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-s3bucket.zip | n/a | | [sqs\_alarms](#module\_sqs\_alarms) | ../../modules/alarms-sqs | n/a | -| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | -| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.4/terraform-sqs.zip | n/a | +| [sqs\_letter\_updates](#module\_sqs\_letter\_updates) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip | n/a | +| [sqs\_supplier\_allocator](#module\_sqs\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/3.1.5/terraform-sqs.zip | n/a | | [supplier\_allocator](#module\_supplier\_allocator) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a | | [supplier\_ssl](#module\_supplier\_ssl) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.26/terraform-ssl.zip | n/a | | [update\_letter\_queue](#module\_update\_letter\_queue) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.29/terraform-lambda.zip | n/a |