Skip to content

Commit

Permalink
Merge pull request #677 from UKHSA-Internal/bug/ingestion-lambda-imag…
Browse files Browse the repository at this point in the history
…e-permissions/CDD-1690

Provide lambda function ARN instead of role ARN to ingestion ECR
  • Loading branch information
A-Ashiq authored May 3, 2024
2 parents 16e7679 + 15d4dcd commit c0cf5e9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion terraform/20-app/ecr.tf
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ module "ecr_ingestion" {
repository_name = "${local.prefix}-ingestion"
repository_read_access_arns = ["arn:aws:iam::${var.assume_account_id}:root"]
repository_read_write_access_arns = ["arn:aws:iam::${var.tools_account_id}:root"]
repository_lambda_read_access_arns = [module.lambda_ingestion.lambda_role_arn]
repository_lambda_read_access_arns = [module.lambda_ingestion.lambda_function_arn]

create_lifecycle_policy = true
repository_lifecycle_policy = local.standard_ecr_lifecycle_policy
Expand Down

0 comments on commit c0cf5e9

Please sign in to comment.