Skip to content

Commit f7ed5e3

Browse files
ci(kernel-e2e): add id-token: write for JFrog OIDC exchange
setup-poetry runs setup-jfrog, which exchanges a GitHub OIDC token for a JFrog access token to reach the internal PyPI mirror. That needs id-token: write on the job, which was missing — the labelled preview run failed at setup-poetry with "ACTIONS_ID_TOKEN_REQUEST_TOKEN: unbound variable". Declared at both workflow scope and on run-kernel-e2e directly: a job-level permissions block fully overrides workflow scope, so the redundancy is intentional. Co-authored-by: Isaac Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>
1 parent e567403 commit f7ed5e3

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

.github/workflows/kernel-e2e.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,12 @@ on:
4040

4141
permissions:
4242
contents: read
43+
# id-token: write is needed by .github/actions/setup-jfrog (OIDC
44+
# exchange with JFrog for the connector's PyPI mirror). Declared
45+
# workflow-wide so the labelled-PR / merge-queue jobs that invoke
46+
# setup-poetry inherit it. Individual jobs still scope down to the
47+
# minimum they actually use (checks: write etc.).
48+
id-token: write
4349

4450
# Cancel in-flight kernel-e2e runs on PR pushes — the warehouse state
4551
# is shared with code-coverage.yml so we already pay this cost there.
@@ -193,6 +199,10 @@ jobs:
193199
permissions:
194200
contents: read
195201
checks: write
202+
# OIDC token exchange with JFrog inside setup-poetry. A job-level
203+
# permissions block fully overrides workflow-level, so this must
204+
# be redeclared here even though the workflow declares it too.
205+
id-token: write
196206
env:
197207
DATABRICKS_SERVER_HOSTNAME: ${{ secrets.DATABRICKS_HOST }}
198208
DATABRICKS_HTTP_PATH: ${{ secrets.TEST_PECO_WAREHOUSE_HTTP_PATH }}

0 commit comments

Comments
 (0)