steve/databricks-cli
steve
/
databricks-cli
mirror of https://github.com/databricks/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Ignore databricks_permissions resource when loading Terraform state (#291) 

## Changes

The databricks_permissions resource may be generated if a bundle
resource includes a `permissions` block. There's no need to incorporate
details from the materialization into the bundle configuration struct.

## Tests

Confirmed that this fixes `bricks bundle run` when dealing with a bundle
with permission configuration.
This commit is contained in:
Pieter Noordhuis 2023-03-29 21:14:52 +02:00 committed by GitHub
parent 87207bba78
commit 0ea0e81c8a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
bundle/deploy/terraform/convert.go
View File

@ -193,6 +193,8 @@ func TerraformToBundle(state *tfjson.State, config *config.Root) error {
cur := config.Resources.Experiments[resource.Name] cur := config.Resources.Experiments[resource.Name]
conv(tmp, &cur) conv(tmp, &cur)
config.Resources.Experiments[resource.Name] = cur config.Resources.Experiments[resource.Name] = cur
case "databricks_permissions":
// Ignore; no need to pull these back into the configuration.
default: default:
return fmt.Errorf("missing mapping for %s", resource.Type) return fmt.Errorf("missing mapping for %s", resource.Type)
} }