[Feature] Skip calling Read after Create/Update operations for notebooks

databricks / terraform-provider-databricks

Databricks Terraform Provider

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

Other

456 stars 393 forks source link

[Feature] Skip calling Read after Create/Update operations for notebooks #4173

Open alexott opened 3 weeks ago

alexott commented 3 weeks ago

Changes

It's based on #4190.

It was found that the import API returns object_id as a result of its execution, so we don't really need to call get-status to fill all attributes. This should help when we import a large number of notebooks, i.e., when applying exported resources.

Tests

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

alexott commented 2 weeks ago

@mgyucht I've opened #4190 with changes in common so it will be easier to review. This PR will be used only for notebook changes.

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: 4173
Commit SHA: 7a54068be05b4c3bbe6b6c3dc9a410cf7f0dc997

Checks will be approved automatically on success.

eng-dev-ecosystem-bot commented 2 days ago

Test Details: go/deco-tests/11913582700