Skip to content

Commit 5ae6e36

Browse files
authored
Merge branch 'master' into feature/VED-1044-create-cloudwatch-dashboard-parameterised-tf
2 parents 4a1b91d + 2a1a63c commit 5ae6e36

36 files changed

Lines changed: 1363 additions & 735 deletions

README.md

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -276,3 +276,20 @@ run a different set of tests. To do this:
276276
Please note that this project requires that all commits are verified using a GPG key.
277277
To set up a GPG key please follow the instructions specified here:
278278
https://docs.github.com/en/authentication/managing-commit-signature-verification
279+
280+
## AWS configuration: getting credentials for AWS federated user accounts
281+
282+
If you need to run commands that interact with AWS resources e.g. running a terraform plan against a dev environment locally
283+
then you will need to configure AWS credentials.
284+
285+
Once you have been granted access, the `Access Keys` section within the AWS Access Portal will present you with several
286+
options. It is _recommended_ to use `Option 2: Add a profile to your AWS credentials file`.
287+
288+
This is because the way that Python unittests using `moto` have been implemented is brittle and cannot handle other methods
289+
such as IAM Identity Centre SSO. In future, we should consider following [moto recommendations](https://docs.getmoto.org/en/latest/docs/getting_started.html#how-do-i-avoid-tests-from-mutating-my-real-infrastructure)
290+
to ensure our tests are authentication type agnostic and are fully robust.
291+
292+
If you _are_ using another option, such as SSO, and want to run unit tests then you will need to:
293+
294+
- Add dummy values for `AWS_ACCESS_KEY_ID` and `AWS_SECRET_ACCESS_KEY` to the .env file in the Lambda dir you are testing.
295+
- Ensure those values are set before running the test, i.e. using the standard setup with direnv and a .envrc file

infrastructure/account/.terraform.lock.hcl

Lines changed: 16 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

infrastructure/instance/.terraform.lock.hcl

Lines changed: 16 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

infrastructure/instance/ack_lambda.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ resource "aws_ecr_repository" "ack_lambda_repository" {
1818
# Module for building and pushing Docker image to ECR
1919
module "ack_processor_docker_image" {
2020
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
21-
version = "8.5.0"
21+
version = "8.7.0"
2222
docker_file_path = "./ack_backend/Dockerfile"
2323
create_ecr_repo = false
2424
ecr_repo = aws_ecr_repository.ack_lambda_repository.name

infrastructure/instance/batch_processor_filter_lambda.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ resource "aws_ecr_repository" "batch_processor_filter_lambda_repository" {
1717
# Module for building and pushing Docker image to ECR
1818
module "batch_processor_filter_docker_image" {
1919
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
20-
version = "8.5.0"
20+
version = "8.7.0"
2121
docker_file_path = "./batch_processor_filter/Dockerfile"
2222

2323
create_ecr_repo = false

infrastructure/instance/delta.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ resource "aws_ecr_repository" "delta_lambda_repository" {
1616

1717
module "delta_docker_image" {
1818
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
19-
version = "8.5.0"
19+
version = "8.7.0"
2020
docker_file_path = "./delta_backend/Dockerfile"
2121

2222
create_ecr_repo = false

infrastructure/instance/ecs_batch_processor_config.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ resource "aws_ecr_repository" "processing_repository" {
2727
# Build and Push Docker Image to ECR (Reusing the existing module)
2828
module "processing_docker_image" {
2929
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
30-
version = "8.5.0"
30+
version = "8.7.0"
3131

3232
create_ecr_repo = false
3333
docker_file_path = "./recordprocessor/Dockerfile"

infrastructure/instance/file_name_processor.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ resource "aws_ecr_repository" "file_name_processor_lambda_repository" {
1717
# Module for building and pushing Docker image to ECR
1818
module "file_processor_docker_image" {
1919
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
20-
version = "8.5.0"
20+
version = "8.7.0"
2121
docker_file_path = "./filenameprocessor/Dockerfile"
2222

2323
create_ecr_repo = false

infrastructure/instance/forwarder_lambda.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ resource "aws_ecr_repository" "forwarder_lambda_repository" {
1717
# Module for building and pushing Docker image to ECR
1818
module "forwarding_docker_image" {
1919
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
20-
version = "8.5.0"
20+
version = "8.7.0"
2121
docker_file_path = "./recordforwarder/Dockerfile"
2222

2323
create_ecr_repo = false

infrastructure/instance/id_sync_lambda.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ resource "aws_ecr_repository" "id_sync_lambda_repository" {
1919
# Module for building and pushing Docker image to ECR
2020
module "id_sync_docker_image" {
2121
source = "terraform-aws-modules/lambda/aws//modules/docker-build"
22-
version = "8.5.0"
22+
version = "8.7.0"
2323
docker_file_path = "./id_sync/Dockerfile"
2424
create_ecr_repo = false
2525
ecr_repo = aws_ecr_repository.id_sync_lambda_repository.name

0 commit comments

Comments
 (0)