Merge pull request #82 from data-science-ucsb/aws-ecr-deploy

Set up push to AWS ECR
data-science-ucsb · Nov 1, 2023 · e632d50 · e632d50
2 parents c299c63 + 442ada2
commit e632d50
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -29,12 +29,16 @@ jobs:
           drive: docker
           install: true
 
+      - name: Setup AWS ECR details
+        uses: aws-actions/configure-aws-credentials@v1
+        with:
+          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
+          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
+          aws-region: us-east-1
+
       - name: Login to AWS ECR
-        run: |
-          aws configure set aws_access_key_id ${{ secrets.AWS_ACCESS_KEY_ID }}
-          aws configure set aws_secret_access_key ${{ secrets.AWS_SECRET_ACCESS_KEY }}
-          aws configure set default.region us-east-1
-          aws ecr get-login-password --region us-east-1 | docker login --username AWS --password-stdin 380357184503.dkr.ecr.us-east-1.amazonaws.com
+        id: login-pf-aws-ecr
+        uses: aws-actions/amazon-ecr-login@v1
 
       - name: Build and push to AWS ECR
         run: |