-
Notifications
You must be signed in to change notification settings - Fork 426
[Internal] Bump Go SDK to latest release + Generate TF Structs #4645
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.
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
Conversation
…ricks into go-sdk-bump-14april
…ricks into go-sdk-bump-14april
Co-authored-by: Omer Lachish <[email protected]>
Co-authored-by: Omer Lachish <[email protected]>
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
## Release v1.73.0 ### New Features and Improvements * Add a new settings resource `databricks_disable_legacy_dbfs_setting` ([#4605](#4605)) * Support for `lifetime` attribute in `databricks_service_principal_secret` resource ([#4633](#4633)). * Mark `databricks_sql_dashaboard` as deprecated ([#4641](#4641)) ### Bug Fixes * Increase the default HTTP timeout to 65 seconds. ### Documentation * Document `user_api_scopes` in `databricks_app` resource and data sources ([#4614](#4614)) * Document new fields in `databricks_model_serving` resource ([#4615](#4615)) ### Exporter * Add export of `databricks_mws_network_connectivity_config` and `databricks_mws_ncc_private_endpoint_rule` ([#4613](#4613)) ### Internal Changes * Refactor `databricks_mws_workspaces` resource and datasource, as well as `databricks_mws_ncc_binding`, to use the Go SDK ([#4633](#4633)). * Add `TestMwsAccGcpWorkspaces` and `TestMwsAccGcpByovpcWorkspaces` to flaky test ([#4624](#4624)). * Bump Go SDK to latest release ([#4645](#4645))
attrs["workspace_status"] = attrs["workspace_status"].SetComputed() | ||
attrs["workspace_status"] = attrs["workspace_status"].SetOptional() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tanmay-db I think that it's wrong.
attrs["vpc_status"] = attrs["vpc_status"].SetComputed() | ||
attrs["vpc_status"] = attrs["vpc_status"].SetOptional() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tanmay-db I think that it's wrong.
Changes
Bump Go SDK to latest release + Generate TF Structs
Tests
CI Tests
make test
run locallydocs/
folderinternal/acceptance