Skip to content

Commit

Permalink
upgrade to v2 of ecr login action
Browse files Browse the repository at this point in the history
  • Loading branch information
philerooski committed Nov 14, 2024
1 parent 40dfd8d commit d1b22d2
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions .github/workflows/upload-and-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -193,14 +193,9 @@ jobs:
- name: Login to Amazon ECR
id: login-ecr
uses: aws-actions/amazon-ecr-login@v1
with:
mask-password: 'false'

- name: Get ECR secret names
id: ecr
run: |
usernameKey=docker_username_$(echo ${{ steps.login-ecr.outputs.registry }} | tr '.-' _)
echo "username-key=$usernameKey" >> $GITHUB_OUTPUT
passwordKey=docker_password_$(echo ${{ steps.login-ecr.outputs.registry }} | tr '.-' _)
echo "password-key=$passwordKey" >> $GITHUB_OUTPUT
- uses: actions/checkout@v3

- name: Set up Docker Buildx
Expand All @@ -215,10 +210,11 @@ jobs:
file: ${{ matrix.dockerfile }}
cache-from: type=local,src=/tmp/.buildx-cache
cache-to: type=local,dest=/tmp/.buildx-cache

outputs:
ecr-registry: ${{ steps.login-ecr.outputs.registry }}
ecr-username: ${{ steps.login-ecr.outputs[steps.ecr.outputs.username-key] }}
ecr-password: ${{ steps.login-ecr.outputs[steps.ecr.outputs.password-key] }}
ecr-username: ${{ steps.login-ecr.outputs["docker_username_914833433684.dkr.ecr.us-east-1.amazonaws.com"] }}
ecr-password: ${{ steps.login-ecr.outputs["docker_password_914833433684.dkr.ecr.us-east-1.amazonaws.com"] }}

glue-unit-tests:
name: Run Pytest unit tests for AWS glue
Expand Down

0 comments on commit d1b22d2

Please sign in to comment.