Skip to content

Commit 277bf45

Browse files
CCM-14149: Letter Preview Placeholder
1 parent 709fed1 commit 277bf45

4 files changed

Lines changed: 28 additions & 12 deletions

File tree

infrastructure/terraform/components/app/module_letter_preview_renderer_lambda.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module "letter_preview_renderer_lambda" {
1414
kms_key_arn = module.kms.key_arn
1515

1616
package_type = "Image"
17-
image_uri = "${var.aws_account_id}.dkr.ecr.${var.region}.amazonaws.com/${var.letter_preview_renderer_ecr_repo}:main-${substr(var.commit_id, 0, 7)}"
17+
image_uri = "${var.aws_account_id}.dkr.ecr.${var.region}.amazonaws.com/${var.letter_preview_renderer_ecr_repo}:${local.csi}-latest"
1818
image_repository_names = [var.letter_preview_renderer_ecr_repo]
1919

2020
memory = 1024

infrastructure/terraform/components/app/pre.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ npm run generate-dependencies --workspaces --if-present
1313
export AWS_REGION="${AWS_REGION:-${TF_VAR_region:-}}"
1414
export AWS_ACCOUNT_ID="${AWS_ACCOUNT_ID:-${TF_VAR_aws_account_id:-}}"
1515
export ECR_REPO="${ECR_REPO:-${TF_VAR_letter_preview_renderer_ecr_repo:-nhs-notify-main-acct}}"
16+
export CSI="${CSI:-${TF_VAR_project:-}-${TF_VAR_environment:-}-${TF_VAR_component:-}}"
17+
CSI="${CSI//_/}"
18+
export CSI
1619

1720
npm run lambda-build --workspaces --if-present
1821

infrastructure/terraform/components/sandbox/pre.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,13 @@ if [ "${ACTION}" == "apply" ]; then
2525

2626
npm run generate-dependencies --workspaces --if-present
2727

28+
export AWS_REGION="${AWS_REGION:-${TF_VAR_region:-}}"
29+
export AWS_ACCOUNT_ID="${AWS_ACCOUNT_ID:-${TF_VAR_aws_account_id:-}}"
30+
export ECR_REPO="${ECR_REPO:-${TF_VAR_letter_preview_renderer_ecr_repo:-nhs-notify-main-acct}}"
31+
export CSI="${CSI:-${TF_VAR_project:-}-${TF_VAR_environment:-}-${TF_VAR_component:-}}"
32+
CSI="${CSI//_/}"
33+
export CSI
34+
2835
npm run lambda-build --workspaces --if-present
2936

3037
lambdas/layers/pdfjs/build.sh

lambdas/letter-preview-renderer/docker.sh

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ set -euo pipefail
1010
: "${AWS_ACCOUNT_ID:?AWS_ACCOUNT_ID is required}"
1111
: "${AWS_REGION:?AWS_REGION is required}"
1212
: "${ECR_REPO:?ECR_REPO is required}"
13+
: "${CSI:?CSI is required}"
1314

1415
# Authenticate Docker with AWS ECR using an ephemeral login token.
1516
aws ecr get-login-password --region "${AWS_REGION}" | docker login --username AWS --password-stdin "${AWS_ACCOUNT_ID}".dkr.ecr."${AWS_REGION}".amazonaws.com
@@ -19,16 +20,17 @@ if [ -n "${GHCR_LOGIN_USER:-}" ] && [ -n "${GHCR_LOGIN_TOKEN:-}" ]; then
1920
echo "${GHCR_LOGIN_TOKEN}" | docker login ghcr.io --username "${GHCR_LOGIN_USER}" --password-stdin
2021
fi
2122

22-
# Resolve the image tag: prefer a GitHub tag, otherwise use main-<short-sha>.
23+
# Resolve git references for image tags.
2324
GIT_SHA=$(git rev-parse --short HEAD)
24-
if [ "${GITHUB_REF_TYPE:-}" = "tag" ] && [ -n "${GITHUB_REF_NAME:-}" ]; then
25-
IMAGE_TAG="${GITHUB_REF_NAME}"
26-
else
27-
IMAGE_TAG="main-${GIT_SHA}"
28-
fi
2925

30-
# Compose the full ECR image reference.
31-
ECR_IMAGE="${AWS_ACCOUNT_ID}.dkr.ecr.${AWS_REGION}.amazonaws.com/${ECR_REPO}:${IMAGE_TAG}"
26+
# Namespace tags by CSI to avoid cross-environment collisions.
27+
IMAGE_TAG_LATEST="${CSI}-latest"
28+
IMAGE_TAG_COMMIT="${CSI}-${GIT_SHA}"
29+
30+
# Compose the full ECR image references.
31+
ECR_REPO_URI="${AWS_ACCOUNT_ID}.dkr.ecr.${AWS_REGION}.amazonaws.com/${ECR_REPO}"
32+
ECR_IMAGE_LATEST="${ECR_REPO_URI}:${IMAGE_TAG_LATEST}"
33+
ECR_IMAGE_COMMIT="${ECR_REPO_URI}:${IMAGE_TAG_COMMIT}"
3234

3335
# Allow an override for the base image used in the Docker build.
3436
BASE_IMAGE_ARG=${BASE_IMAGE:-ghcr.io/nhsdigital/nhs-notify/letter-renderer-node-22:latest}
@@ -37,8 +39,12 @@ BASE_IMAGE_ARG=${BASE_IMAGE:-ghcr.io/nhsdigital/nhs-notify/letter-renderer-node-
3739
docker build \
3840
-f docker/lambda/Dockerfile \
3941
--build-arg BASE_IMAGE="${BASE_IMAGE_ARG}" \
40-
-t "${ECR_IMAGE}" \
42+
-t "${ECR_IMAGE_LATEST}" \
4143
.
4244

43-
# Push the image to ECR.
44-
docker push "${ECR_IMAGE}"
45+
# Apply additional tag containing the commit identifier.
46+
docker tag "${ECR_IMAGE_LATEST}" "${ECR_IMAGE_COMMIT}"
47+
48+
# Push the image tags to ECR.
49+
docker push "${ECR_IMAGE_LATEST}"
50+
docker push "${ECR_IMAGE_COMMIT}"

0 commit comments

Comments
 (0)