Fix updating owners

databricks · Jan 31, 2024 · 7932c78 · 7932c78
1 parent 510fd60
commit 7932c78
Show file tree

Hide file tree

Showing 8 changed files with 39 additions and 0 deletions.
diff --git a/catalog/resource_catalog.go b/catalog/resource_catalog.go
@@ -148,6 +148,10 @@ func ResourceCatalog() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			updateCatalogRequest.Owner = ""
 			ci, err := w.Catalogs.Update(ctx, updateCatalogRequest)
 

diff --git a/catalog/resource_external_location.go b/catalog/resource_external_location.go
@@ -108,6 +108,10 @@ func ResourceExternalLocation() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			updateExternalLocationRequest.Owner = ""
 			_, err = w.ExternalLocations.Update(ctx, updateExternalLocationRequest)
 			if err != nil {

diff --git a/catalog/resource_metastore.go b/catalog/resource_metastore.go
@@ -138,6 +138,11 @@ func ResourceMetastore() *schema.Resource {
 						return err
 					}
 				}
+
+				if !d.HasChangeExcept("owner") {
+					return nil
+				}
+
 				update.Owner = ""
 				_, err := acc.Metastores.Update(ctx, catalog.AccountsUpdateMetastore{
 					MetastoreId:   d.Id(),
@@ -171,6 +176,11 @@ func ResourceMetastore() *schema.Resource {
 						return err
 					}
 				}
+
+				if !d.HasChangeExcept("owner") {
+					return nil
+				}
+
 				update.Owner = ""
 				_, err := w.Metastores.Update(ctx, update)
 				if err != nil {

diff --git a/catalog/resource_schema.go b/catalog/resource_schema.go
@@ -99,6 +99,10 @@ func ResourceSchema() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			updateSchemaRequest.Owner = ""
 			schema, err := w.Schemas.Update(ctx, updateSchemaRequest)
 			if err != nil {

diff --git a/catalog/resource_share.go b/catalog/resource_share.go
@@ -237,6 +237,10 @@ func ResourceShare() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			err = NewSharesAPI(ctx, c).update(d.Id(), ShareUpdates{
 				Updates: changes,
 			})

diff --git a/catalog/resource_storage_credential.go b/catalog/resource_storage_credential.go
@@ -152,6 +152,11 @@ func ResourceStorageCredential() *schema.Resource {
 						return err
 					}
 				}
+
+				if !d.HasChangeExcept("owner") {
+					return nil
+				}
+
 				update.Owner = ""
 				_, err := acc.StorageCredentials.Update(ctx, catalog.AccountsUpdateStorageCredential{
 					CredentialInfo:        &update,

diff --git a/catalog/resource_volume.go b/catalog/resource_volume.go
@@ -93,6 +93,10 @@ func ResourceVolume() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			updateVolumeRequestContent.Owner = ""
 			v, err := w.Volumes.Update(ctx, updateVolumeRequestContent)
 			if err != nil {

diff --git a/sharing/resource_recipient.go b/sharing/resource_recipient.go
@@ -85,6 +85,10 @@ func ResourceRecipient() *schema.Resource {
 				}
 			}
 
+			if !d.HasChangeExcept("owner") {
+				return nil
+			}
+
 			updateRecipientRequest.Owner = ""
 			err = w.Recipients.Update(ctx, updateRecipientRequest)
 			if err != nil {