Skip to content

Pass through OIDC token env variable to Terraform #3113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 25, 2025
Merged

Conversation

andrewnester
Copy link
Contributor

Changes

Pass through OIDC token env variable to Terraform

Why

This allows OIDC auth to correctly work in DABs, otherwise it fails with

Error: cannot create job: failed during request visitor: env-oidc auth: missing env var "DATABRICKS_OIDC_TOKEN". Config: host=*** client_id=***. Env: DATABRICKS_HOST, DATABRICKS_CLIENT_ID

Tests

Added unit test

@pietern
Copy link
Contributor

pietern commented Jun 24, 2025

@renaudhartert-db Can you take a look as well?

@pietern pietern requested a review from renaudhartert-db June 24, 2025 13:48
@andrewnester andrewnester merged commit 8b6919f into main Jun 25, 2025
21 of 23 checks passed
@andrewnester andrewnester deleted the feature/dabs-oidc branch June 25, 2025 14:42
deco-sdk-tagging bot added a commit that referenced this pull request Jun 25, 2025
## Release v0.257.0

### Bundles
* Improve error message for host mismatch between bundle and profile used ([#3100](#3100))
* Remove support for deprecated `experimental/pydabs` config, use `experimental/python` instead. See [Configuration in Python
](https://docs.databricks.com/dev-tools/bundles/python). ([#3102](#3102))
* Pass through OIDC token env variable to Terraform ([#3113](#3113))

* The `default-python` template now prompts if you want to use serverless compute (default to `yes`) ([#3051](#3051)).

### API Changes
* Removed `databricks custom-llms` command group.
* Added `databricks ai-builder` command group.
* Added `databricks feature-store` command group.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants