Skip to content

Commit 98d0577

Browse files
authored
DRIVERS-3232 Move ECR login to before Assuming Team Role (#92)
1 parent 4e6efbd commit 98d0577

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

setup/action.yml

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,17 @@ inputs:
2323
runs:
2424
using: composite
2525
steps:
26+
# Docker login must be done before input role login so we remain logged in for S3 upload.
27+
- name: configure aws credentials for ECR
28+
uses: aws-actions/configure-aws-credentials@e3dd6a429d7300a6a4c196c26e071d42e0343502 # v4
29+
with:
30+
role-to-assume: ${{ inputs.ecr_role_arn }}
31+
role-session-name: release-session
32+
aws-region: ${{ inputs.ecr_region }}
33+
- name: Log in to ECR
34+
uses: aws-actions/amazon-ecr-login@062b18b96a7aff071d4dc91bc00c4c1a7945b076 # v2
35+
with:
36+
registries: "${{ inputs.ecr_registry_id }}"
2637
- name: configure aws credentials
2738
uses: aws-actions/configure-aws-credentials@e3dd6a429d7300a6a4c196c26e071d42e0343502 # v4
2839
with:
@@ -40,14 +51,4 @@ runs:
4051
id: setup
4152
run: ${{ github.action_path }}/setup.sh
4253
env:
43-
AWS_SECRET_ID: ${{ inputs.aws_secret_id }}
44-
- name: configure aws credentials for ECR
45-
uses: aws-actions/configure-aws-credentials@e3dd6a429d7300a6a4c196c26e071d42e0343502 # v4
46-
with:
47-
role-to-assume: ${{ inputs.ecr_role_arn }}
48-
role-session-name: release-session
49-
aws-region: ${{ inputs.ecr_region }}
50-
- name: Log in to ECR
51-
uses: aws-actions/amazon-ecr-login@062b18b96a7aff071d4dc91bc00c4c1a7945b076 # v2
52-
with:
53-
registries: "${{ inputs.ecr_registry_id }}"
54+
AWS_SECRET_ID: ${{ inputs.aws_secret_id }}

0 commit comments

Comments
 (0)