Skip to content

Add warning when variable interpolation is used for auth fields - #2399

Merged
shreyas-goenka merged 7 commits into
mainfrom
auth-warning
Feb 27, 2025
Merged

Add warning when variable interpolation is used for auth fields#2399
shreyas-goenka merged 7 commits into
mainfrom
auth-warning

Conversation

@shreyas-goenka

@shreyas-goenkashreyas-goenka commented Feb 27, 2025

Copy link
Copy Markdown
Contributor

Changes

This PR adds a warning which gives users clear guidance when they try to use variable interpolation for an auth field.

Tests

Modify existing acceptance test.

Comment threadbundle/config/validate/interpolation_in_auth_config.go
@denik

Copy link
Copy Markdown
Contributor

Related: #2095

Comment threadacceptance/bundle/validate/interpolation-in-auth-field/databricks.yml Outdated
@shreyas-goenka
shreyas-goenka added this pull request to the merge queueFeb 27, 2025
Merged via the queue into main with commit bc299caFeb 27, 2025
@shreyas-goenka
shreyas-goenka deleted the auth-warning branch February 27, 2025 16:07
Comment threadbundle/config/validate/interpolation_in_auth_config.go
Comment threadbundle/config/validate/interpolation_in_auth_config.go
HariGS-DB pushed a commit to HariGS-DB/cli that referenced this pull request Feb 28, 2025
…bricks#2399)
## Changes
This PR adds a warning which gives users clear guidance when they try to
use variable interpolation for an auth field.
## Tests
Modify existing acceptance test.
github-merge-queueBot pushed a commit that referenced this pull request Mar 3, 2025
## Why
Addresses post merge feedback from
#2399 (comment)
## Tests
N/A
andrewnester added a commit that referenced this pull request Mar 5, 2025
CLI:
* Upgrade Go SDK to 0.59.0 ([#2425](#2425)).
Bundles:
* Added a warning when `config` section is used in apps ([#2416](#2416)).
* Switch to use GET workspaces-files/{name} instead of workspace/export for state files to avoid 10MB limit ([#2423](#2423)).
* Use schema field for pipeline in builtin template ([#2347](#2347)).
* Add warning when variable interpolation is used for auth fields ([#2399](#2399)).
* Add warning when include is used in config files other than databricks.yml ([#2389](#2389)).
* Add support for schemas in deployment bind/unbind commands ([#2406](#2406)).
* Do not modify/create .gitignore in bundle root ([#2429](#2429)).
* Raise an error when there are multiple local libraries with the same basename used ([#2382](#2382)).
* Upgrade TF provider to 1.68.0 ([#2426](#2426)).
API Changes:
* Changed `databricks experiments log-inputs` command with new required argument order.
* Added `databricks genie get-space` command.
* Added `databricks providers list-provider-share-assets` command.
* Changed `databricks shares update-permissions` command return type to become non-empty.
OpenAPI commit e5c870006a536121442cfd2441bdc8a5fb76ae1e (2025-03-03)
github-merge-queueBot pushed a commit that referenced this pull request Mar 5, 2025
CLI:
* Upgrade Go SDK to 0.59.0
([#2425](#2425)).
Bundles:
* Added a warning when `config` section is used in apps
([#2416](#2416)).
* Switch to use GET workspaces-files/{name} instead of workspace/export
for state files to avoid 10MB limit
([#2423](#2423)).
* Use schema field for pipeline in builtin template
([#2347](#2347)).
* Add warning when variable interpolation is used for auth fields
([#2399](#2399)).
* Add warning when include is used in config files other than
databricks.yml ([#2389](#2389)).
* Add support for schemas in deployment bind/unbind commands
([#2406](#2406)).
* Do not modify/create .gitignore in bundle root
([#2429](#2429)).
* Raise an error when there are multiple local libraries with the same
basename used ([#2382](#2382)).
* Upgrade TF provider to 1.68.0
([#2426](#2426)).
API Changes:
* Changed `databricks experiments log-inputs` command with new required
argument order.
* Added `databricks genie get-space` command.
* Added `databricks providers list-provider-share-assets` command.
* Changed `databricks shares update-permissions` command return type to
become non-empty.
OpenAPI commit e5c870006a536121442cfd2441bdc8a5fb76ae1e (2025-03-03)
denik pushed a commit that referenced this pull request May 20, 2026
## Changes
This PR adds a warning which gives users clear guidance when they try to
use variable interpolation for an auth field.
## Tests
Modify existing acceptance test.
denik pushed a commit that referenced this pull request May 20, 2026
## Why
Addresses post merge feedback from
#2399 (comment)
## Tests
N/A
denik pushed a commit that referenced this pull request May 20, 2026
CLI:
* Upgrade Go SDK to 0.59.0
([#2425](#2425)).
Bundles:
* Added a warning when `config` section is used in apps
([#2416](#2416)).
* Switch to use GET workspaces-files/{name} instead of workspace/export
for state files to avoid 10MB limit
([#2423](#2423)).
* Use schema field for pipeline in builtin template
([#2347](#2347)).
* Add warning when variable interpolation is used for auth fields
([#2399](#2399)).
* Add warning when include is used in config files other than
databricks.yml ([#2389](#2389)).
* Add support for schemas in deployment bind/unbind commands
([#2406](#2406)).
* Do not modify/create .gitignore in bundle root
([#2429](#2429)).
* Raise an error when there are multiple local libraries with the same
basename used ([#2382](#2382)).
* Upgrade TF provider to 1.68.0
([#2426](#2426)).
API Changes:
* Changed `databricks experiments log-inputs` command with new required
argument order.
* Added `databricks genie get-space` command.
* Added `databricks providers list-provider-share-assets` command.
* Changed `databricks shares update-permissions` command return type to
become non-empty.
OpenAPI commit e5c870006a536121442cfd2441bdc8a5fb76ae1e (2025-03-03)
Sign up for freeto 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.

4 participants

@shreyas-goenka@denik@pietern@andrewnester