Skip to content

Commit bc6f24f

Browse files
CCM-12858: fixes for terraform
1 parent 425c342 commit bc6f24f

File tree

3 files changed

+7
-19
lines changed

3 files changed

+7
-19
lines changed

infrastructure/terraform/components/dl/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,11 @@ No requirements.
3636

3737
| Name | Source | Version |
3838
|------|--------|---------|
39+
| <a name="module_core_notifier"></a> [core\_notifier](#module\_core\_notifier) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip | n/a |
3940
| <a name="module_kms"></a> [kms](#module\_kms) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-kms.zip | n/a |
4041
| <a name="module_lambda_apim_key_generation"></a> [lambda\_apim\_key\_generation](#module\_lambda\_apim\_key\_generation) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip | n/a |
4142
| <a name="module_lambda_lambda_apim_refresh_token"></a> [lambda\_lambda\_apim\_refresh\_token](#module\_lambda\_lambda\_apim\_refresh\_token) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip | n/a |
4243
| <a name="module_mesh_poll"></a> [mesh\_poll](#module\_mesh\_poll) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip | n/a |
43-
| <a name="module_pdm_uploader"></a> [pdm\_uploader](#module\_pdm\_uploader) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip | n/a |
4444
| <a name="module_s3bucket_cf_logs"></a> [s3bucket\_cf\_logs](#module\_s3bucket\_cf\_logs) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-s3bucket.zip | n/a |
4545
| <a name="module_s3bucket_letters"></a> [s3bucket\_letters](#module\_s3bucket\_letters) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-s3bucket.zip | n/a |
4646
| <a name="module_s3bucket_static_assets"></a> [s3bucket\_static\_assets](#module\_s3bucket\_static\_assets) | https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-s3bucket.zip | n/a |

infrastructure/terraform/components/dl/module_lambda_core_notifier.tf

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
module "pdm_uploader" {
1+
module "core_notifier" {
22
source = "https://github.com/NHSDigital/nhs-notify-shared-modules/releases/download/v2.0.24/terraform-lambda.zip"
33

44
function_name = "core-notifier"
5-
description = "A function to upload documents to PDM"
5+
description = "A function to send messages to core Notify when a PDM resource is available"
66

77
aws_account_id = var.aws_account_id
88
component = local.component
@@ -15,7 +15,7 @@ module "pdm_uploader" {
1515
kms_key_arn = module.kms.key_arn
1616

1717
iam_policy_document = {
18-
body = data.aws_iam_policy_document.pdm_uploader_lambda.json
18+
body = data.aws_iam_policy_document.core_notifier_lambda.json
1919
}
2020

2121
function_s3_bucket = local.acct.s3_buckets["lambda_function_artefacts"]["id"]
@@ -43,7 +43,7 @@ module "pdm_uploader" {
4343
}
4444
}
4545

46-
data "aws_iam_policy_document" "pdm_uploader_lambda" {
46+
data "aws_iam_policy_document" "core_notifier_lambda" {
4747
statement {
4848
sid = "AllowSSMParam"
4949
effect = "Allow"
@@ -59,19 +59,6 @@ data "aws_iam_policy_document" "pdm_uploader_lambda" {
5959
]
6060
}
6161

62-
statement {
63-
sid = "AllowS3Get"
64-
effect = "Allow"
65-
66-
actions = [
67-
"s3:GetObject"
68-
]
69-
70-
resources = [
71-
"${module.s3bucket_letters.arn}/*"
72-
]
73-
}
74-
7562
statement {
7663
sid = "KMSPermissions"
7764
effect = "Allow"
@@ -87,7 +74,7 @@ data "aws_iam_policy_document" "pdm_uploader_lambda" {
8774
}
8875

8976
statement {
90-
sid = "SQSPermissionsUploadToPdmQueue"
77+
sid = "SQSPermissionsUploadToCoreNotifierQueue"
9178
effect = "Allow"
9279

9380
actions = [

tests/playwright/constants/backend-constants.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ export const CSI = `nhs-${ENV}-dl`;
1111
export const MESH_POLL_LAMBDA_NAME = `${CSI}-mesh-poll`;
1212
export const TTL_CREATE_LAMBDA_NAME = `${CSI}-ttl-create`;
1313
export const TTL_POLL_LAMBDA_NAME = `${CSI}-ttl-poll`;
14+
export const CORE_NOTIFIER_LAMBDA_NAME = `${CSI}-core-notifier`;
1415

1516
// Queue Names
1617
export const TTL_QUEUE_NAME = `${CSI}-ttl-queue`;

0 commit comments

Comments
 (0)