Skip to content

Commit 925c145

Browse files
author
Alexander Melnyk
committed
fix: add entire ARN role instead of account and role name
1 parent e0b3b58 commit 925c145

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

.github/workflows/publish_layer.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
stage: "BETA"
6767
artefact-name: "cdk-layer-artefact"
6868
secrets:
69-
target-account: ${{ secrets.LAYERS_BETA_ACCOUNT }}
69+
target-account-role: arn:aws:iam::${{ secrets.LAYERS_BETA_ACCOUNT }}:role/${{ secrets.AWS_GITHUB_OIDC_ROLE }}
7070

7171
deploy-prod:
7272
needs:
@@ -76,4 +76,4 @@ jobs:
7676
stage: "PROD"
7777
artefact-name: "cdk-layer-artefact"
7878
secrets:
79-
target-account: ${{ secrets.LAYERS_PROD_ACCOUNT }}
79+
target-account-role: arn:aws:iam::${{ secrets.LAYERS_PROD_ACCOUNT }}:role/${{ secrets.AWS_GITHUB_OIDC_ROLE }}

.github/workflows/reusable_deploy_layer_stack.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ on:
1414
required: true
1515
type: string
1616
secrets:
17-
target-account:
17+
target-account-role:
1818
required: true
1919

2020
jobs:
@@ -57,7 +57,7 @@ jobs:
5757
uses: aws-actions/configure-aws-credentials@v1
5858
with:
5959
aws-region: ${{ matrix.region }}
60-
role-to-assume: arn:aws:iam::${{ secrets.target-account }}:role/${{ secrets.AWS_GITHUB_OIDC_ROLE }}
60+
role-to-assume: ${{ secrets.target-account-role }}
6161
- name: Setup Node.js
6262
uses: actions/setup-node@v2
6363
with:

0 commit comments

Comments
 (0)