Skip to content
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

[Doc] Clarify that databricks_token and databricks_obo_token could be used only with workspace-level provider #4480

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

alexott
Copy link
Contributor

@alexott alexott commented Feb 8, 2025

Changes

Resolves #4458

Tests

  • make test run locally
  • relevant change in docs/ folder
  • covered with integration tests in internal/acceptance
  • using Go SDK
  • using TF Plugin Framework

@alexott alexott requested review from a team as code owners February 8, 2025 03:56
@alexott alexott requested review from hectorcast-db and removed request for a team February 8, 2025 03:56
@alexott alexott temporarily deployed to test-trigger-is February 8, 2025 03:56 — with GitHub Actions Inactive
@alexott alexott temporarily deployed to test-trigger-is February 8, 2025 03:57 — with GitHub Actions Inactive
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 4480
  • Commit SHA: f8a114a858c872d7822a6113b217852ea9c37f75

Checks will be approved automatically on success.

@alexott alexott added this pull request to the merge queue Feb 11, 2025
Merged via the queue into main with commit bd38f68 Feb 11, 2025
13 checks passed
@alexott alexott deleted the issue-4458 branch February 11, 2025 15:05
deco-sdk-tagging bot added a commit that referenced this pull request Feb 13, 2025
## Release v1.66.0

### New Features and Improvements

 * Add multipart permissions to `databricks_aws_unity_catalog_policy` data source ([#4440](#4440)).

### Bug Fixes
 * Fixed an issue where reordering objects in a (pluginfw) Share wouldn’t update properly unless other changes were made ([#4481](#4481)).

 * Suppress `options.pem_private_key_expiration_epoch_sec` attribute for databricks_connection ([#4474](#4474)).

### Documentation

 * Add an example for Databricks Apps permissions ([#4475](#4475)).
 * Add explanation of timeouts to the troubleshooting guide ([#4482](#4482)).
 * Clarify that `databricks_token` and `databricks_obo_token` could be used only with workspace-level provider ([#4480](#4480)).

### Exporter

 * Refactor UC, SQL and SCIM objects into separate files ([#4477](#4477)).

### Internal Changes

 * Remove incorrectly working integration test `TestAccLibraryUpdateTransitionFromPluginFw` ([#4487](#4487)).
@lkanerv
Copy link

lkanerv commented Mar 26, 2025

Ty for this doc update!

To shorten the feedback loops for users, would also suggest that databricks_token return a 400 status error with "Only a workspace-level provider" can be used if the provider level is incorrect, since the current error of Error: cannot create token: failed to unmarshal response body: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK isn't very helpful.

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.

[FEATURE] Allow to use databricks_token resource to issue account-level SCIM token
3 participants