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.

Sign up for GitHub

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

Jump to bottom

[Doc] add clarification note for databricks_grants #4546

Merged
merged 3 commits into from
Mar 11, 2025
Merged

[Doc] add clarification note for databricks_grants #4546

merged 3 commits into from
Mar 11, 2025
+10 −7

Conversation

nkvuong
Copy link
Contributor

@nkvuong nkvuong commented Mar 4, 2025

Changes

  • Add a note on how databricks_grants work with MANAGE permission

Tests

  • relevant change in docs/ folder

@nkvuong nkvuong requested review from a team as code owners March 4, 2025 08:12
@nkvuong nkvuong requested review from rauchy and removed request for a team March 4, 2025 08:12
@nkvuong nkvuong temporarily deployed to test-trigger-is March 4, 2025 08:12 — with GitHub Actions Inactive
@nkvuong nkvuong temporarily deployed to test-trigger-is March 4, 2025 08:14 — with GitHub Actions Inactive
@nkvuong nkvuong enabled auto-merge March 4, 2025 08:45
@nkvuong nkvuong disabled auto-merge March 4, 2025 08:47
@nkvuong nkvuong enabled auto-merge March 4, 2025 08:54
@nkvuong nkvuong force-pushed the doc/grant_clarification branch from c9e50ad to 67653d7 Compare March 5, 2025 07:10
@nkvuong nkvuong temporarily deployed to test-trigger-is March 5, 2025 07:10 — with GitHub Actions Inactive
@nkvuong nkvuong temporarily deployed to test-trigger-is March 5, 2025 07:12 — with GitHub Actions Inactive
@alexott alexott temporarily deployed to test-trigger-is March 11, 2025 18:16 — with GitHub Actions Inactive
@github-actions GitHub Actions
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: 4546
  • Commit SHA: 090aed81144020a2bb3cb9e5afab97df32e22df9

Checks will be approved automatically on success.

@alexott alexott temporarily deployed to test-trigger-is March 11, 2025 18:17 — with GitHub Actions Inactive
@nkvuong nkvuong added this pull request to the merge queue Mar 11, 2025
Merged via the queue into main with commit 5346452 Mar 11, 2025
13 checks passed
@nkvuong nkvuong deleted the doc/grant_clarification branch March 11, 2025 19:01
deco-sdk-tagging bot added a commit that referenced this pull request Mar 12, 2025
@deco-sdk-tagging
[Release] Release v1.70.0
b9ccd98 
## Release v1.70.0

### Bug Fixes

 * Fix creation of `databricks_storage_credential` and `databricks_credential` resources on GCP with isolation mode ([#4563](#4563))
 * Handle auto-enabled errors with `databricks_system_schema` [#4547](#4547)
 * Skip Read after Create in `databricks_secret_acl` to avoid errors([#4548](#4548)).

### Documentation

 * Document `amazon_bedrock_config.instance_profile_arn` in `databricks_model_serving` ([#4549](#4549))
 * Document management of permissions of `databricks_budget_policy` resource ([#4561](#4561))
 * Document `budget_policy_id` in `databricks_app` resource and data sources ([#4557](#4557))
 * Add a note on how `databricks_grants` work with `MANAGE` permission [#4546](#4546)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexott alexott alexott approved these changes

@rauchy rauchy Awaiting requested review from rauchy rauchy is a code owner automatically assigned from databricks/eng-plat-auto-exp-reviewers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nkvuong @alexott