Skip to content

ci(snowflake): allow tests to use WIF auth - #1954

Merged
colin-k-rogers merged 2 commits into
mainfrom
ragesh/snowflake-wif-setup
May 19, 2026
Merged

ci(snowflake): allow tests to use WIF auth#1954
colin-k-rogers merged 2 commits into
mainfrom
ragesh/snowflake-wif-setup

Conversation

@ragesh-g

@ragesh-g ragesh-g commented May 19, 2026

Copy link
Copy Markdown
Contributor

resolves N/A

Problem

The integration-tests-snowflake CI job can't authenticate via Workload Identity Federation, blocking the WIF functional test in #1316.

Solution

  • Adds id-token: write to the snowflake job so the runner exposes the OIDC env vars (ACTIONS_ID_TOKEN_REQUEST_URL / ACTIONS_ID_TOKEN_REQUEST_TOKEN) to the test process.
  • Adds SNOWFLAKE_TEST_WIF_USER env var pointing at a dedicated WIF service user (separate from SNOWFLAKE_TEST_USER because Snowflake WIF requires TYPE = SERVICE, which can't have password auth).

Currently unused. Consumed by #1316; landing first so the consuming PR has the infra it needs (pull_request_target workflows load YAML from main).

Testing

[gw3] [ 50%] PASSED tests/functional/auth_tests/test_workload_identity_federation_oidc.py::TestSnowflakeWorkloadIdentityFederation::test_snowflake_wif_basic_functionality

Ran the test against the PR #1316

Checklist

  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@cla-bot cla-bot Bot added the cla:yes The PR author has signed the CLA label May 19, 2026
@ragesh-g
ragesh-g force-pushed the ragesh/snowflake-wif-setup branch from b911b9c to 0393e93 Compare May 19, 2026 15:21
@ragesh-g
ragesh-g temporarily deployed to dbt-snowflake May 19, 2026 15:25 — with GitHub Actions Inactive
@ragesh-g
ragesh-g force-pushed the ragesh/snowflake-wif-setup branch from 0393e93 to 7efa8ea Compare May 19, 2026 16:41
@ragesh-g
ragesh-g temporarily deployed to dbt-snowflake May 19, 2026 16:42 — with GitHub Actions Inactive
@ragesh-g
ragesh-g force-pushed the ragesh/snowflake-wif-setup branch from 7efa8ea to 105a556 Compare May 19, 2026 16:48
@ragesh-g
ragesh-g temporarily deployed to dbt-snowflake May 19, 2026 16:49 — with GitHub Actions Inactive
@ragesh-g ragesh-g changed the title ci(snowflake): add OIDC token minting for WIF ci(snowflake): enable OIDC for WIF tests May 19, 2026
@ragesh-g
ragesh-g marked this pull request as ready for review May 19, 2026 17:33
@ragesh-g
ragesh-g requested a review from a team as a code owner May 19, 2026 17:33
@github-actions github-actions Bot added the community A PR, or an issue with a PR, from a community member label May 19, 2026
@ragesh-g
ragesh-g force-pushed the ragesh/snowflake-wif-setup branch from 105a556 to 6bfc693 Compare May 19, 2026 17:37
@ragesh-g ragesh-g changed the title ci(snowflake): enable OIDC for WIF tests ci(snowflake): allow tests to use WIF auth May 19, 2026
@colin-k-rogers
colin-k-rogers merged commit 6d4f8f0 into main May 19, 2026
9 checks passed
@colin-k-rogers
colin-k-rogers deleted the ragesh/snowflake-wif-setup branch May 19, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla:yes The PR author has signed the CLA community A PR, or an issue with a PR, from a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants