Skip to content

Commit 3eadde6

Browse files
authored
[Internal] Bump Go SDK to v0.72 & Generate Latest TF Structs (#4777)
## Changes <!-- Summary of your changes that are easy to understand --> ## Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] `make test` run locally - [ ] relevant change in `docs/` folder - [ ] covered with integration tests in `internal/acceptance` - [ ] using Go SDK - [ ] using TF Plugin Framework --------- Co-authored-by: Omer Lachish <[email protected]>
1 parent 8565981 commit 3eadde6

File tree

69 files changed

+13126
-7309
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

69 files changed

+13126
-7309
lines changed

.codegen/_openapi_sha

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
2cee201b2e8d656f7306b2f9ec98edfa721e9829
1+
a8f547d3728fba835fbdda301e846829c5cbbef5

.gitattributes

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,16 @@ docs/data-sources/budget_policies.md linguist-generated=true
1515
docs/data-sources/budget_policy.md linguist-generated=true
1616
docs/data-sources/database_instance.md linguist-generated=true
1717
docs/data-sources/database_instances.md linguist-generated=true
18+
docs/data-sources/quality_monitor_v2.md linguist-generated=true
19+
docs/data-sources/quality_monitors_v2.md linguist-generated=true
1820
docs/data-sources/recipient_federation_policies.md linguist-generated=true
1921
docs/data-sources/recipient_federation_policy.md linguist-generated=true
2022
docs/data-sources/workspace_network_option.md linguist-generated=true
2123
docs/resources/account_network_policy.md linguist-generated=true
2224
docs/resources/alert_v2.md linguist-generated=true
2325
docs/resources/budget_policy.md linguist-generated=true
2426
docs/resources/database_instance.md linguist-generated=true
27+
docs/resources/quality_monitor_v2.md linguist-generated=true
2528
docs/resources/recipient_federation_policy.md linguist-generated=true
2629
docs/resources/workspace_network_option.md linguist-generated=true
2730
internal/providers/common/common.go linguist-generated=true
@@ -47,6 +50,9 @@ internal/providers/pluginfw/products/budget_policy/resource_budget_policy.go lin
4750
internal/providers/pluginfw/products/database_instance/data_database_instance.go linguist-generated=true
4851
internal/providers/pluginfw/products/database_instance/data_database_instances.go linguist-generated=true
4952
internal/providers/pluginfw/products/database_instance/resource_database_instance.go linguist-generated=true
53+
internal/providers/pluginfw/products/quality_monitor_v2/data_quality_monitor_v2.go linguist-generated=true
54+
internal/providers/pluginfw/products/quality_monitor_v2/data_quality_monitors_v2.go linguist-generated=true
55+
internal/providers/pluginfw/products/quality_monitor_v2/resource_quality_monitor_v2.go linguist-generated=true
5056
internal/providers/pluginfw/products/recipient_federation_policy/data_recipient_federation_policies.go linguist-generated=true
5157
internal/providers/pluginfw/products/recipient_federation_policy/data_recipient_federation_policy.go linguist-generated=true
5258
internal/providers/pluginfw/products/recipient_federation_policy/resource_recipient_federation_policy.go linguist-generated=true
@@ -71,6 +77,8 @@ internal/providers/pluginfw/tfschema/single_nested_attribute.go linguist-generat
7177
internal/providers/pluginfw/tfschema/single_nested_block.go linguist-generated=true
7278
internal/providers/pluginfw/tfschema/string_attribute.go linguist-generated=true
7379
internal/providers/pluginfw/tfschema/struct_to_schema.go linguist-generated=true
80+
internal/service/aibuilder_tf/legacy_model.go linguist-generated=true
81+
internal/service/aibuilder_tf/model.go linguist-generated=true
7482
internal/service/apps_tf/legacy_model.go linguist-generated=true
7583
internal/service/apps_tf/model.go linguist-generated=true
7684
internal/service/billing_tf/legacy_model.go linguist-generated=true
@@ -83,6 +91,8 @@ internal/service/compute_tf/legacy_model.go linguist-generated=true
8391
internal/service/compute_tf/model.go linguist-generated=true
8492
internal/service/dashboards_tf/legacy_model.go linguist-generated=true
8593
internal/service/dashboards_tf/model.go linguist-generated=true
94+
internal/service/database_tf/legacy_model.go linguist-generated=true
95+
internal/service/database_tf/model.go linguist-generated=true
8696
internal/service/files_tf/legacy_model.go linguist-generated=true
8797
internal/service/files_tf/model.go linguist-generated=true
8898
internal/service/iam_tf/legacy_model.go linguist-generated=true
@@ -99,6 +109,8 @@ internal/service/pipelines_tf/legacy_model.go linguist-generated=true
99109
internal/service/pipelines_tf/model.go linguist-generated=true
100110
internal/service/provisioning_tf/legacy_model.go linguist-generated=true
101111
internal/service/provisioning_tf/model.go linguist-generated=true
112+
internal/service/qualitymonitorv2_tf/legacy_model.go linguist-generated=true
113+
internal/service/qualitymonitorv2_tf/model.go linguist-generated=true
102114
internal/service/serving_tf/legacy_model.go linguist-generated=true
103115
internal/service/serving_tf/model.go linguist-generated=true
104116
internal/service/settings_tf/legacy_model.go linguist-generated=true

.github/workflows/tagging.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jobs:
2121
steps:
2222
- name: Generate GitHub App Token
2323
id: generate-token
24-
uses: actions/create-github-app-token@v1
24+
uses: actions/create-github-app-token@v2
2525
with:
2626
app-id: ${{ secrets.DECO_SDK_TAGGING_APP_ID }}
2727
private-key: ${{ secrets.DECO_SDK_TAGGING_PRIVATE_KEY }}
@@ -49,4 +49,3 @@ jobs:
4949
GITHUB_REPOSITORY: ${{ github.repository }}
5050
run: |
5151
python tagging.py
52-

NEXT_CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
* Support for `uc_securable` resource in `databricks_app` ([#4767](https://github.com/databricks/terraform-provider-databricks/pull/4767))
1010
* Add `schema_id` to the output of `databricks_schema` resource ([#4773](https://github.com/databricks/terraform-provider-databricks/pull/4773)).
11+
* Bump the Go SDK to v0.72.0 ([#4777](https://github.com/databricks/terraform-provider-databricks/pull/4777)).
1112

1213
### Bug Fixes
1314

catalog/permissions/permissions.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,15 @@ func NewUnityCatalogPermissionsAPI(ctx context.Context, m any) UnityCatalogPermi
2626
return UnityCatalogPermissionsAPI{client, ctx}
2727
}
2828

29-
func (a UnityCatalogPermissionsAPI) GetPermissions(securable catalog.SecurableType, name string) (list *catalog.PermissionsList, err error) {
29+
func (a UnityCatalogPermissionsAPI) GetPermissions(securable catalog.SecurableType, name string) (list *catalog.GetPermissionsResponse, err error) {
3030
if securable.String() == "share" {
3131
sharePermissions, err := a.client.Shares.SharePermissions(a.context, sharing.SharePermissionsRequest{
3232
Name: name,
3333
})
3434
if err != nil {
3535
return nil, err
3636
}
37-
list = &catalog.PermissionsList{
37+
list = &catalog.GetPermissionsResponse{
3838
PrivilegeAssignments: make([]catalog.PrivilegeAssignment, len(sharePermissions.PrivilegeAssignments)),
3939
}
4040
for i, pa := range sharePermissions.PrivilegeAssignments {
@@ -45,7 +45,7 @@ func (a UnityCatalogPermissionsAPI) GetPermissions(securable catalog.SecurableTy
4545
}
4646
return list, nil
4747
}
48-
list, err = a.client.Grants.GetBySecurableTypeAndFullName(a.context, securable, name)
48+
list, err = a.client.Grants.GetBySecurableTypeAndFullName(a.context, securable.String(), name)
4949
return
5050
}
5151

@@ -67,24 +67,24 @@ func (a UnityCatalogPermissionsAPI) UpdatePermissions(securable catalog.Securabl
6767
}
6868
_, err := a.client.Grants.Update(a.context, catalog.UpdatePermissions{
6969
Changes: diff,
70-
SecurableType: securable,
70+
SecurableType: securable.String(),
7171
FullName: name,
7272
})
7373
return err
7474
}
7575

76-
func (a UnityCatalogPermissionsAPI) WaitForUpdate(timeout time.Duration, securable catalog.SecurableType, name string, desired catalog.PermissionsList, diff func(*catalog.PermissionsList, catalog.PermissionsList) []catalog.PermissionsChange) error {
76+
func (a UnityCatalogPermissionsAPI) WaitForUpdate(timeout time.Duration, securable catalog.SecurableType, name string, desired []catalog.PrivilegeAssignment, diff func([]catalog.PrivilegeAssignment, []catalog.PrivilegeAssignment) []catalog.PermissionsChange) error {
7777
return retry.RetryContext(a.context, timeout, func() *retry.RetryError {
7878
current, err := a.GetPermissions(securable, name)
7979
if err != nil {
8080
return retry.NonRetryableError(err)
8181
}
82-
log.Printf("[DEBUG] Permissions for %s-%s are: %v", securable.String(), name, current)
83-
if diff(current, desired) == nil {
82+
log.Printf("[DEBUG] Permissions for %s-%s are: %v", securable.String(), name, current.PrivilegeAssignments)
83+
if diff(current.PrivilegeAssignments, desired) == nil {
8484
return nil
8585
}
8686
return retry.RetryableError(
87-
fmt.Errorf("permissions for %s-%s are %v, but have to be %v", securable.String(), name, current, desired),
87+
fmt.Errorf("permissions for %s-%s are %v, but have to be %v", securable.String(), name, current.PrivilegeAssignments, desired),
8888
)
8989
})
9090
}

catalog/resource_grant.go

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,17 @@ import (
1515
)
1616

1717
// diffPermissionsForPrincipal returns an array of catalog.PermissionsChange of this permissions list with `diff` privileges removed
18-
func diffPermissionsForPrincipal(principal string, desired catalog.PermissionsList, existing catalog.PermissionsList) (diff []catalog.PermissionsChange) {
18+
func diffPermissionsForPrincipal(principal string, desired []catalog.PrivilegeAssignment, existing []catalog.PrivilegeAssignment) (diff []catalog.PermissionsChange) {
1919
// diffs change sets for principal
2020
configured := map[string]*schema.Set{}
21-
for _, v := range desired.PrivilegeAssignments {
21+
for _, v := range desired {
2222
if v.Principal == principal {
2323
configured[v.Principal] = permissions.SliceToSet(v.Privileges)
2424
}
2525
}
2626
// existing permissions that needs removal for principal
2727
remote := map[string]*schema.Set{}
28-
for _, v := range existing.PrivilegeAssignments {
28+
for _, v := range existing {
2929
if v.Principal == principal {
3030
remote[v.Principal] = permissions.SliceToSet(v.Privileges)
3131
}
@@ -66,25 +66,25 @@ func diffPermissionsForPrincipal(principal string, desired catalog.PermissionsLi
6666
}
6767

6868
// replacePermissionsForPrincipal merges removal diff of existing permissions on the platform
69-
func replacePermissionsForPrincipal(a permissions.UnityCatalogPermissionsAPI, securable string, name string, principal string, list catalog.PermissionsList) error {
69+
func replacePermissionsForPrincipal(a permissions.UnityCatalogPermissionsAPI, securable string, name string, principal string, list catalog.GetPermissionsResponse) error {
7070
securableType := permissions.Mappings.GetSecurableType(securable)
7171
existing, err := a.GetPermissions(securableType, name)
7272
if err != nil {
7373
return err
7474
}
75-
err = a.UpdatePermissions(securableType, name, diffPermissionsForPrincipal(principal, list, *existing))
75+
err = a.UpdatePermissions(securableType, name, diffPermissionsForPrincipal(principal, list.PrivilegeAssignments, existing.PrivilegeAssignments))
7676
if err != nil {
7777
return err
7878
}
79-
return a.WaitForUpdate(1*time.Minute, securableType, name, list, func(current *catalog.PermissionsList, desired catalog.PermissionsList) []catalog.PermissionsChange {
80-
return diffPermissionsForPrincipal(principal, desired, *current)
79+
return a.WaitForUpdate(1*time.Minute, securableType, name, list.PrivilegeAssignments, func(current []catalog.PrivilegeAssignment, desired []catalog.PrivilegeAssignment) []catalog.PermissionsChange {
80+
return diffPermissionsForPrincipal(principal, desired, current)
8181
})
8282
}
8383

8484
// filterPermissionsForPrincipal extracts permissions for the given principal and transforms to permissions.UnityCatalogPrivilegeAssignment to match Schema
85-
func filterPermissionsForPrincipal(in catalog.PermissionsList, principal string) (*permissions.UnityCatalogPrivilegeAssignment, error) {
85+
func filterPermissionsForPrincipal(in []catalog.PrivilegeAssignment, principal string) (*permissions.UnityCatalogPrivilegeAssignment, error) {
8686
grantsForPrincipal := []permissions.UnityCatalogPrivilegeAssignment{}
87-
for _, v := range in.PrivilegeAssignments {
87+
for _, v := range in {
8888
privileges := []string{}
8989
if v.Principal == principal {
9090
for _, p := range v.Privileges {
@@ -166,7 +166,7 @@ func ResourceGrant() common.Resource {
166166
}
167167
principal := d.Get("principal").(string)
168168
privileges := permissions.SetToSlice(d.Get("privileges").(*schema.Set))
169-
var grants = catalog.PermissionsList{
169+
var grants = catalog.GetPermissionsResponse{
170170
PrivilegeAssignments: []catalog.PrivilegeAssignment{
171171
{
172172
Principal: principal,
@@ -192,7 +192,7 @@ func ResourceGrant() common.Resource {
192192
if err != nil {
193193
return err
194194
}
195-
grantsForPrincipal, err := filterPermissionsForPrincipal(*grants, principal)
195+
grantsForPrincipal, err := filterPermissionsForPrincipal(grants.PrivilegeAssignments, principal)
196196
if err != nil {
197197
return err
198198
}
@@ -216,7 +216,7 @@ func ResourceGrant() common.Resource {
216216
return err
217217
}
218218
privileges := permissions.SetToSlice(d.Get("privileges").(*schema.Set))
219-
var grants = catalog.PermissionsList{
219+
var grants = catalog.GetPermissionsResponse{
220220
PrivilegeAssignments: []catalog.PrivilegeAssignment{
221221
{
222222
Principal: principal,
@@ -241,7 +241,7 @@ func ResourceGrant() common.Resource {
241241
return err
242242
}
243243
unityCatalogPermissionsAPI := permissions.NewUnityCatalogPermissionsAPI(ctx, c)
244-
return replacePermissionsForPrincipal(unityCatalogPermissionsAPI, securable, name, principal, catalog.PermissionsList{})
244+
return replacePermissionsForPrincipal(unityCatalogPermissionsAPI, securable, name, principal, catalog.GetPermissionsResponse{})
245245
},
246246
}
247247
}

0 commit comments

Comments
 (0)