From 889eacf2371ebdb3711ec1baf5f6607a369ab13f Mon Sep 17 00:00:00 2001
From: mbadibanga <83414778+mbadibanga@users.noreply.github.com>
Date: Sat, 9 Oct 2021 18:07:49 +0200
Subject: [PATCH] Docker & ECR fixes

Please add Docker installation as requirement, the build-containers script fails due to it

"aws ecr" used old syntax AWS CLI version 1 which does not work anymore
---
 content/prerequisites/bootstrapsh.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/content/prerequisites/bootstrapsh.md b/content/prerequisites/bootstrapsh.md
index 9e055be..3fb91cd 100644
--- a/content/prerequisites/bootstrapsh.md
+++ b/content/prerequisites/bootstrapsh.md
@@ -142,7 +142,7 @@ cat > ~/environment/scripts/build-containers <<-"EOF"
 CRYSTAL_ECR_REPO=$(jq < cfn-output.json -r '.CrystalEcrRepo')
 NODEJS_ECR_REPO=$(jq < cfn-output.json -r '.NodeJSEcrRepo')
 
-$(aws ecr get-login --no-include-email)
+aws ecr get-login-password --region ${AWS_REGION} | docker login --username AWS --password-stdin $CRYSTAL_ECR_REPO
 
 docker build -t crystal-service ecsdemo-crystal
 docker tag crystal-service:latest $CRYSTAL_ECR_REPO:vanilla