[PRM-658] Increase default lambda memory from 512MB to 5308MB (#566)

PedroSoaresNHS · robg-test · web-flow · commit 5bd4d9af28bd · 2026-01-29T14:52:54.000Z
Co-authored-by: Robert Gaskin &lt;106234256+robg-test@users.noreply.github.com&gt;
diff --git a/infrastructure/lambda-bulk-upload-metadata-preprocessor.tf b/infrastructure/lambda-bulk-upload-metadata-preprocessor.tf
@@ -23,7 +23,6 @@ module "bulk_upload_metadata_preprocessor_lambda" {
   is_gateway_integration_needed  = false
   is_invoked_from_gateway        = false
   lambda_timeout                 = 900
-  memory_size                    = 1769
   reserved_concurrent_executions = local.bulk_upload_lambda_concurrent_limit
 }
 
diff --git a/infrastructure/lambda-bulk-upload-metadata-processor.tf b/infrastructure/lambda-bulk-upload-metadata-processor.tf
@@ -3,7 +3,6 @@ module "bulk-upload-metadata-processor-lambda" {
   name           = "BulkUploadMetadataProcessor"
   handler        = "handlers.bulk_upload_metadata_processor_handler.lambda_handler"
   lambda_timeout = 900
-  memory_size    = 1769
 
   iam_role_policy_documents = [
     module.ndr-bulk-staging-store.s3_read_policy_document,
diff --git a/infrastructure/lambda-bulk-upload-metadata.tf b/infrastructure/lambda-bulk-upload-metadata.tf
@@ -3,7 +3,6 @@ module "bulk-upload-metadata-lambda" {
   name           = "BulkUploadMetadataLambda"
   handler        = "handlers.bulk_upload_metadata_handler.lambda_handler"
   lambda_timeout = 900
-  memory_size    = 1769
   iam_role_policy_documents = [
     module.ndr-bulk-staging-store.s3_read_policy_document,
     module.ndr-bulk-staging-store.s3_write_policy_document,
diff --git a/infrastructure/lambda-bulk-upload-report.tf b/infrastructure/lambda-bulk-upload-report.tf
@@ -3,7 +3,6 @@ module "bulk-upload-report-lambda" {
   name           = "BulkUploadReportLambda"
   handler        = "handlers.bulk_upload_report_handler.lambda_handler"
   lambda_timeout = 900
-  memory_size    = 1769
   iam_role_policy_documents = [
     module.statistical-reports-store.s3_read_policy_document,
     module.statistical-reports-store.s3_write_policy_document,
diff --git a/infrastructure/lambda-create-doc-ref.tf b/infrastructure/lambda-create-doc-ref.tf
@@ -63,7 +63,6 @@ module "create-doc-ref-lambda" {
   rest_api_id         = aws_api_gateway_rest_api.ndr_doc_store_api.id
   resource_id         = module.document_reference_gateway.gateway_resource_id
   http_methods        = ["POST"]
-  memory_size         = 512
 
   api_execution_arn = aws_api_gateway_rest_api.ndr_doc_store_api.execution_arn
   lambda_environment_variables = {
diff --git a/infrastructure/lambda-document-review-processor.tf b/infrastructure/lambda-document-review-processor.tf
@@ -13,7 +13,6 @@ module "document_review_processor_lambda" {
     aws_iam_policy.ssm_access_policy.policy
   ]
   kms_deletion_window           = var.kms_deletion_window
-  memory_size                   = 512
   is_gateway_integration_needed = false
   is_invoked_from_gateway       = false
   rest_api_id                   = null
diff --git a/infrastructure/lambda-dynamodb-migration.tf b/infrastructure/lambda-dynamodb-migration.tf
@@ -29,7 +29,6 @@ module "migration-dynamodb-lambda" {
   }
 
   lambda_timeout                 = 900
-  memory_size                    = 1024
   reserved_concurrent_executions = contains(["prod"], terraform.workspace) ? 100 : 5
 
   depends_on = [
diff --git a/infrastructure/lambda-generate-document-manifest.tf b/infrastructure/lambda-generate-document-manifest.tf
@@ -45,7 +45,6 @@ module "generate-document-manifest-lambda" {
   handler                  = "handlers.generate_document_manifest_handler.lambda_handler"
   lambda_timeout           = 900
   lambda_ephemeral_storage = 512
-  memory_size              = 1769
   iam_role_policy_documents = [
     module.ndr-document-store.s3_read_policy_document,
     module.ndr-document-store.s3_write_policy_document,
diff --git a/infrastructure/lambda-generate-stitch-record.tf b/infrastructure/lambda-generate-stitch-record.tf
@@ -45,7 +45,6 @@ module "generate-lloyd-george-stitch-lambda" {
   handler                  = "handlers.generate_lloyd_george_stitch_handler.lambda_handler"
   lambda_timeout           = 900
   lambda_ephemeral_storage = 1024
-  memory_size              = 1769
   iam_role_policy_documents = [
     module.ndr-document-store.s3_read_policy_document,
     module.ndr-document-store.s3_write_policy_document,
diff --git a/infrastructure/lambda-get-report-by-ods.tf b/infrastructure/lambda-get-report-by-ods.tf
@@ -64,7 +64,6 @@ module "get-report-by-ods-lambda" {
   rest_api_id         = aws_api_gateway_rest_api.ndr_doc_store_api.id
   resource_id         = module.get-report-by-ods-gateway.gateway_resource_id
   http_methods        = ["GET"]
-  memory_size         = 1769
   lambda_timeout      = 900
   api_execution_arn   = aws_api_gateway_rest_api.ndr_doc_store_api.execution_arn
   lambda_environment_variables = {
diff --git a/infrastructure/lambda-migration-dynamodb-segment.tf b/infrastructure/lambda-migration-dynamodb-segment.tf
@@ -3,7 +3,6 @@ module "migration-dynamodb-segment-lambda" {
   name           = "MigrationDynamodbSegment"
   handler        = "handlers.migration_dynamodb_segment_handler.lambda_handler"
   lambda_timeout = 900
-  memory_size    = 1792
   iam_role_policy_documents = [
     module.migration-dynamodb-segment-store.s3_read_policy_document,
     module.migration-dynamodb-segment-store.s3_write_policy_document,
diff --git a/infrastructure/lambda-report-orchestration.tf b/infrastructure/lambda-report-orchestration.tf
@@ -3,7 +3,7 @@ module "report-orchestration-lambda" {
   name           = "reportOrchestration"
   handler        = "handlers.report_orchestration_handler.lambda_handler"
   lambda_timeout = 900
-  memory_size    = 1769
+  memory_size    = 5038
 
   iam_role_policy_documents = [
     module.bulk_upload_report_dynamodb_table.dynamodb_read_policy_document,
@@ -25,4 +25,5 @@ module "report-orchestration-lambda" {
   depends_on = [
     module.bulk_upload_report_dynamodb_table
   ]
-}
+}
+
diff --git a/infrastructure/lambda-search-patient.tf b/infrastructure/lambda-search-patient.tf
@@ -64,7 +64,6 @@ module "search-patient-details-lambda" {
   rest_api_id         = aws_api_gateway_rest_api.ndr_doc_store_api.id
   resource_id         = module.search-patient-details-gateway.gateway_resource_id
   http_methods        = ["GET"]
-  memory_size         = 1769
   lambda_environment_variables = {
     APPCONFIG_APPLICATION          = module.ndr-app-config.app_config_application_id
     APPCONFIG_ENVIRONMENT          = module.ndr-app-config.app_config_environment_id
diff --git a/infrastructure/lambda-token.tf b/infrastructure/lambda-token.tf
@@ -46,7 +46,6 @@ module "create-token-lambda" {
     module.create-token-gateway,
     module.ndr-app-config
   ]
-  memory_size = 1769
 }
 
 module "create_token-alarm" {
diff --git a/infrastructure/modules/lambda/README.md b/infrastructure/modules/lambda/README.md
@@ -113,7 +113,7 @@ module "lambda" {
 | <a name="input_lambda_environment_variables"></a> [lambda\_environment\_variables](#input\_lambda\_environment\_variables) | Map of environment variables to set in the Lambda function. | `map(string)` | `{}` | no |
 | <a name="input_lambda_ephemeral_storage"></a> [lambda\_ephemeral\_storage](#input\_lambda\_ephemeral\_storage) | Amount of ephemeral storage (in MB) to allocate to the Lambda function. | `number` | `512` | no |
 | <a name="input_lambda_timeout"></a> [lambda\_timeout](#input\_lambda\_timeout) | Function timeout in seconds. | `number` | `30` | no |
-| <a name="input_memory_size"></a> [memory\_size](#input\_memory\_size) | Amount of memory to allocate to the Lambda function (in MB). | `number` | `512` | no |
+| <a name="input_memory_size"></a> [memory\_size](#input\_memory\_size) | Amount of memory to allocate to the Lambda function (in MB). | `number` | `5038` | no |
 | <a name="input_name"></a> [name](#input\_name) | Unique name for the Lambda function. | `string` | n/a | yes |
 | <a name="input_persistent_workspaces"></a> [persistent\_workspaces](#input\_persistent\_workspaces) | A list of workspaces that require persistent logs | `list(string)` | <pre>[<br/>  "ndr-dev",<br/>  "ndr-test",<br/>  "pre-prod",<br/>  "prod"<br/>]</pre> | no |
 | <a name="input_reserved_concurrent_executions"></a> [reserved\_concurrent\_executions](#input\_reserved\_concurrent\_executions) | The number of concurrent execution allowed for lambda. A value of 0 will stop lambda from running, and -1 removes any concurrency limitations. Default to -1. | `number` | `-1` | no |
diff --git a/infrastructure/modules/lambda/variable.tf b/infrastructure/modules/lambda/variable.tf
@@ -76,7 +76,7 @@ variable "lambda_ephemeral_storage" {
 variable "memory_size" {
   description = "Amount of memory to allocate to the Lambda function (in MB)."
   type        = number
-  default     = 512
+  default     = 5308
 }
 
 variable "reserved_concurrent_executions" {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,6 @@ module "bulk_upload_metadata_preprocessor_lambda" {`
`23`	`23`	`is_gateway_integration_needed = false`
`24`	`24`	`is_invoked_from_gateway = false`
`25`	`25`	`lambda_timeout = 900`
`26`		`- memory_size = 1769`
`27`	`26`	`reserved_concurrent_executions = local.bulk_upload_lambda_concurrent_limit`
`28`	`27`	`}`
`29`	`28`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@ module "document_review_processor_lambda" {`
`13`	`13`	`aws_iam_policy.ssm_access_policy.policy`
`14`	`14`	`]`
`15`	`15`	`kms_deletion_window = var.kms_deletion_window`
`16`		`- memory_size = 512`
`17`	`16`	`is_gateway_integration_needed = false`
`18`	`17`	`is_invoked_from_gateway = false`
`19`	`18`	`rest_api_id = null`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,6 @@ module "migration-dynamodb-lambda" {`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`lambda_timeout = 900`
`32`		`- memory_size = 1024`
`33`	`32`	`reserved_concurrent_executions = contains(["prod"], terraform.workspace) ? 100 : 5`
`34`	`33`
`35`	`34`	`depends_on = [`