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

[Exporter] Add listing for databricks_permissions so we can emit permissions for tokens #4402

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[Exporter] Add listing for databricks_permissions so we can emit permissions for tokens #4402

wants to merge 1 commit into from
+35 −1

Conversation

alexott
Copy link
Contributor

@alexott alexott commented Jan 16, 2025 •
edited
Loading

Changes

It was a missing functionality when we emitted all permissions on existing objects, but didn't do it for permissions of personal access tokens.

Resolves #4389

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
[Exporter] Add listing for databricks_permissions so we can emit pe…
36688db 
…rmissions for tokens

It was a missing functionality when we emitted all permissions on existing objects, but
didn't do it for permissions of personal access tokens.
@alexott alexott requested review from a team as code owners January 16, 2025 07:54
@alexott alexott requested review from mgyucht and removed request for a team January 16, 2025 07:54
@alexott alexott temporarily deployed to test-trigger-is January 16, 2025 07:54 — 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: 4402
  • Commit SHA: 36688dbe05c91012035772814d79f15c94e42891

Checks will be approved automatically on success.

@alexott alexott added the exporter TF configuration generator label Jan 16, 2025
@alexott alexott temporarily deployed to test-trigger-is January 16, 2025 07:55 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
exporter TF configuration generator
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Exporter] Export permissions for tokens usage
1 participant
@alexott