Skip to content

Commit 863b43a

Browse files
haritamarclaude
andcommitted
ci: grant id-token: write in reusable-workflow callers (CORE-687)
test-all-warehouses.yml and test-release.yml call test-warehouse.yml as a reusable workflow. Per GitHub, id-token: write must be granted by the calling workflow — declaring it only on the called workflow's job is not sufficient. Add the permission to the relevant caller jobs. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 7dfaf66 commit 863b43a

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

.github/workflows/test-all-warehouses.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,12 @@ jobs:
7878

7979
test:
8080
needs: [check-fork-status, approve-fork]
81+
permissions:
82+
contents: read
83+
# Required so the called test-warehouse.yml can mint an OIDC token to
84+
# assume the AWS role; per GitHub, id-token: write must be granted by
85+
# the calling workflow.
86+
id-token: write
8187
if: |
8288
! cancelled() &&
8389
needs.check-fork-status.result == 'success' &&

.github/workflows/test-release.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,12 @@ jobs:
4343
echo "dbt bumped: ${{ steps.bump-tag.outputs.dbt-bumped }}"
4444
4545
validate-upgrade-cli:
46+
permissions:
47+
contents: read
48+
# Required so the called test-warehouse.yml can mint an OIDC token to
49+
# assume the AWS role; per GitHub, id-token: write must be granted by
50+
# the calling workflow.
51+
id-token: write
4652
strategy:
4753
fail-fast: false
4854
matrix:

0 commit comments

Comments
 (0)