container: allow updating storage_pools (#13657)

Signed-off-by: drfaust92 <[email protected]>

Author: DrFaust92

mmv1/third_party/terraform/services/container/node_config.go.tmpl

@@ -470,7 +470,6 @@ func schemaNodeConfig() *schema.Schema {
 
 				"storage_pools": {
 					Type:     schema.TypeList,
-					ForceNew: true,
 					Optional: true,
 					Elem:     &schema.Schema{Type: schema.TypeString},
 					Description: `The list of Storage Pools where boot disks are provisioned.`,

mmv1/third_party/terraform/services/container/resource_container_cluster_test.go.tmpl

@@ -13358,6 +13358,18 @@ func TestAccContainerCluster_storagePoolsWithNodeConfig(t *testing.T) {
 				ImportStateVerify:       true,
 				ImportStateVerifyIgnore: []string{"deletion_protection"},
 			},
+			{
+				Config: testAccContainerCluster_storagePoolsWithNodeConfigUpdate(cluster, location, networkName, subnetworkName),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("google_container_cluster.storage_pools_with_node_config", "node_config.0.storage_pools.#", "0"),
+				),
+			},
+			{
+				ResourceName:            "google_container_cluster.storage_pools_with_node_config",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"deletion_protection"},
+			},
 		},
 	})
 }
@@ -13384,6 +13396,27 @@ resource "google_container_cluster" "storage_pools_with_node_config" {
 `, cluster, location, storagePoolResourceName, networkName, subnetworkName)
 }
 
+func testAccContainerCluster_storagePoolsWithNodeConfigUpdate(cluster, location, networkName, subnetworkName string) string {
+	return fmt.Sprintf(`
+resource "google_container_cluster" "storage_pools_with_node_config" {
+  name     = "%s"
+  location = "%s"
+
+  initial_node_count = 1
+  node_config {
+    machine_type = "c3-standard-4"
+    image_type   = "COS_CONTAINERD"
+    disk_type    = "hyperdisk-balanced"
+  }
+
+  network    = "%s"
+  subnetwork = "%s"
+
+  deletion_protection = false
+}
+`, cluster, location, networkName, subnetworkName)
+}
+
 func TestAccContainerCluster_withAutopilotGcpFilestoreCsiDriver(t *testing.T) {
 	t.Parallel()