azurerm_iothub_device_update_account- sku forces new (#24324)

* `azurerm_iothub_device-update_account`- `sku` forces new now * remove sku test
hashicorp · Jan 3, 2024 · 3bdb031 · 3bdb031
1 parent cfd7490
commit 3bdb031
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 44 deletions.
diff --git a/internal/services/iothub/iothub_device_update_account_resource.go b/internal/services/iothub/iothub_device_update_account_resource.go
@@ -21,9 +21,7 @@ import (
 
 type IotHubDeviceUpdateAccountResource struct{}
 
-var (
-	_ sdk.ResourceWithUpdate = IotHubDeviceUpdateAccountResource{}
-)
+var _ sdk.ResourceWithUpdate = IotHubDeviceUpdateAccountResource{}
 
 type IotHubDeviceUpdateAccountModel struct {
 	Name                       string            `tfschema:"name"`
@@ -58,6 +56,7 @@ func (r IotHubDeviceUpdateAccountResource) Arguments() map[string]*pluginsdk.Sch
 
 		"sku": {
 			Type:     pluginsdk.TypeString,
+			ForceNew: true,
 			Optional: true,
 			Default:  deviceupdates.SKUStandard,
 			ValidateFunc: validation.StringInSlice([]string{
@@ -255,10 +254,6 @@ func (r IotHubDeviceUpdateAccountResource) Update() sdk.ResourceFunc {
 				existing.Properties.PublicNetworkAccess = &publicNetworkAccess
 			}
 
-			if metadata.ResourceData.HasChange("sku") {
-				existing.Properties.Sku = &model.Sku
-			}
-
 			if metadata.ResourceData.HasChange("tags") {
 				existing.Tags = &model.Tags
 			}

diff --git a/internal/services/iothub/iothub_device_update_account_resource_test.go b/internal/services/iothub/iothub_device_update_account_resource_test.go
@@ -120,27 +120,6 @@ func TestAccIotHubDeviceUpdateAccount_publicNetworkAccess(t *testing.T) {
 	})
 }
 
-func TestAccIotHubDeviceUpdateAccount_sku(t *testing.T) {
-	data := acceptance.BuildTestData(t, "azurerm_iothub_device_update_account", "test")
-	r := IotHubDeviceUpdateAccountResource{}
-	data.ResourceTest(t, r, []acceptance.TestStep{
-		{
-			Config: r.sku(data, "Free"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.sku(data, "Standard"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-	})
-}
-
 func TestAccIotHubDeviceUpdateAccount_tags(t *testing.T) {
 	data := acceptance.BuildTestData(t, "azurerm_iothub_device_update_account", "test")
 	r := IotHubDeviceUpdateAccountResource{}
@@ -334,21 +313,6 @@ resource "azurerm_iothub_device_update_account" "test" {
 `, template, data.RandomString, enabled)
 }
 
-func (r IotHubDeviceUpdateAccountResource) sku(data acceptance.TestData, sku string) string {
-	template := r.template(data)
-	return fmt.Sprintf(`
-%s
-
-resource "azurerm_iothub_device_update_account" "test" {
-  name                = "acc-dua-%s"
-  resource_group_name = azurerm_resource_group.test.name
-  location            = azurerm_resource_group.test.location
-
-  sku = "%s"
-}
-`, template, data.RandomString, sku)
-}
-
 func (r IotHubDeviceUpdateAccountResource) tags(data acceptance.TestData) string {
 	template := r.template(data)
 	return fmt.Sprintf(`

diff --git a/website/docs/r/iothub_device_update_account.html.markdown b/website/docs/r/iothub_device_update_account.html.markdown
@@ -47,7 +47,7 @@ The following arguments are supported:
 
 * `public_network_access_enabled` - (Optional) Specifies whether the public network access is enabled for the IoT Hub Device Update Account. Possible values are `true` and `false`. Defaults to `true`.
 
-* `sku` - (Optional) Sku of the IoT Hub Device Update Account. Possible values are `Free` and `Standard`. Defaults to `Standard`.
+* `sku` - (Optional) Sku of the IoT Hub Device Update Account. Possible values are `Free` and `Standard`. Defaults to `Standard`. Changing this forces a new resource to be created.
 
 * `tags` - (Optional) A mapping of tags which should be assigned to the IoT Hub Device Update Account.