Skip to content

Commit 1540681

Browse files
authored
Merge pull request #143 from appirio-tech/dev
fix: docker creds
2 parents e2adc89 + 8dba16d commit 1540681

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

service/build/buildimage.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,10 @@ cp $WORKSPACE/service/target/review-microservice*.jar review-microservice.jar
4040
cp $WORKSPACE/service/src/main/resources/review-service.yaml review-service.yaml
4141

4242
echo "copying dockercfg......"
43-
aws s3 cp "s3://appirio-platform-$ENV/services/common/dockercfg" ~/.dockercfg
43+
#aws s3 cp "s3://appirio-platform-$ENV/services/common/dockercfg" ~/.dockercfg
44+
DOCKER_USER=$(aws ssm get-parameter --name /$ENV_CONFIG/build/dockeruser --with-decryption --output text --query Parameter.Value)
45+
DOCKER_PASSWD=$(aws ssm get-parameter --name /$ENV_CONFIG/build/dockercfg --with-decryption --output text --query Parameter.Value)
46+
echo $DOCKER_PASSWD | docker login -u $DOCKER_USER --password-stdin
4447

4548
echo "Building docker image $DOCKER_REPO/$IMAGE"
4649
docker build -t $DOCKER_REPO/$IMAGE $DEPLOY_DIR

0 commit comments

Comments
 (0)