[Fix] Correct handling of updates for empty comments and `force_destroy` in UC catalog, schema, registered models and volumes

databricks / terraform-provider-databricks

Databricks Terraform Provider

https://registry.terraform.io/providers/databricks/databricks/latest

Other

456 stars 393 forks source link

[Fix] Correct handling of updates for empty comments and `force_destroy` in UC catalog, schema, registered models and volumes #4244

Closed alexott closed 2 days ago

alexott commented 2 days ago

Changes

Force sending of fields when comment is changing to empty value. Also, don't call Update API when the only change is in force_destroy attribute of schemas and catalogs

Resolves #4045

Tests

[x] make test run locally
[ ] relevant change in docs/ folder
[ ] covered with integration tests in internal/acceptance
[ ] relevant acceptance tests are passing
[ ] using Go SDK

github-actions[bot] commented 2 days ago

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: 4244
Commit SHA: 9d8a43bda3f279fbe72a5d68895530071fe09e1f

Checks will be approved automatically on success.

eng-dev-ecosystem-bot commented 2 days ago

Test Details: go/deco-tests/11909297294