Remove MaxItems validation check on Bigtable instance cluster count (#…

…4166) (#7713)

Signed-off-by: Modular Magician <[email protected]>

.changelog/4166.txt

@@ -0,0 +1,3 @@
+```release-note:none
+
+```

google/resource_bigtable_instance.go

@@ -52,7 +52,7 @@ func resourceBigtableInstance() *schema.Resource {
 				Type:        schema.TypeList,
 				Optional:    true,
 				Computed:    true,
-				Description: `A block of cluster configuration options. This can be specified at least once, and up to 4 times.`,
+				Description: `A block of cluster configuration options. This can be specified at least once.`,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
 						"cluster_id": {
@@ -382,13 +382,10 @@ func expandBigtableClusters(clusters []interface{}, instanceID string) []bigtabl
 func resourceBigtableInstanceClusterReorderTypeList(_ context.Context, diff *schema.ResourceDiff, meta interface{}) error {
 	oldCount, newCount := diff.GetChange("cluster.#")
 
-	// simulate Required:true, MinItems:1, MaxItems:4 for "cluster"
+	// simulate Required:true, MinItems:1 for "cluster"
 	if newCount.(int) < 1 {
 		return fmt.Errorf("config is invalid: Too few cluster blocks: Should have at least 1 \"cluster\" block")
 	}
-	if newCount.(int) > 4 {
-		return fmt.Errorf("config is invalid: Too many cluster blocks: No more than 4 \"cluster\" blocks are allowed")
-	}
 
 	// exit early if we're in create (name's old value is nil)
 	n, _ := diff.GetChange("name")

google/resource_bigtable_instance_test.go

@@ -60,10 +60,6 @@ func TestAccBigtableInstance_cluster(t *testing.T) {
 		Providers:    testAccProviders,
 		CheckDestroy: testAccCheckBigtableInstanceDestroyProducer(t),
 		Steps: []resource.TestStep{
-			{
-				Config:      testAccBigtableInstance_clusterMax(instanceName),
-				ExpectError: regexp.MustCompile("config is invalid: Too many cluster blocks: No more than 4 \"cluster\" blocks are allowed"),
-			},
 			{
 				Config: testAccBigtableInstance_cluster(instanceName, 3),
 			},
@@ -250,46 +246,6 @@ resource "google_bigtable_instance" "instance" {
 `, instanceName, instanceName, numNodes, instanceName, numNodes, instanceName, numNodes, instanceName, numNodes)
 }
 
-func testAccBigtableInstance_clusterMax(instanceName string) string {
-	return fmt.Sprintf(`
-resource "google_bigtable_instance" "instance" {
-  name = "%s"
-  cluster {
-    cluster_id   = "%s-a"
-    zone         = "us-central1-a"
-    num_nodes    = 3
-    storage_type = "HDD"
-  }
-  cluster {
-    cluster_id   = "%s-b"
-    zone         = "us-central1-b"
-    num_nodes    = 3
-    storage_type = "HDD"
-  }
-  cluster {
-    cluster_id   = "%s-c"
-    zone         = "us-central1-c"
-    num_nodes    = 3
-    storage_type = "HDD"
-  }
-  cluster {
-    cluster_id   = "%s-d"
-    zone         = "us-central1-f"
-    num_nodes    = 3
-    storage_type = "HDD"
-  }
-  cluster {
-    cluster_id   = "%s-e"
-    zone         = "us-east1-a"
-    num_nodes    = 3
-    storage_type = "HDD"
-  }
-
-  deletion_protection = false
-}
-`, instanceName, instanceName, instanceName, instanceName, instanceName, instanceName)
-}
-
 func testAccBigtableInstance_clusterReordered(instanceName string, numNodes int) string {
 	return fmt.Sprintf(`
 resource "google_bigtable_instance" "instance" {