Added read timeout to bigtable instance (#8948) (#15856)

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

Author: modular-magician

.changelog/8948.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+bigtable: added a read timeout to `google_bigtable_instance`
+```

google/services/bigtable/resource_bigtable_instance.go

@@ -33,6 +33,7 @@ func ResourceBigtableInstance() *schema.Resource {
 		Timeouts: &schema.ResourceTimeout{
 			Create: schema.DefaultTimeout(60 * time.Minute),
 			Update: schema.DefaultTimeout(60 * time.Minute),
+			Read:   schema.DefaultTimeout(60 * time.Minute),
 		},
 
 		CustomizeDiff: customdiff.All(
@@ -257,7 +258,9 @@ func resourceBigtableInstanceRead(d *schema.ResourceData, meta interface{}) erro
 
 	instanceName := d.Get("name").(string)
 
-	instance, err := c.InstanceInfo(ctx, instanceName)
+	ctxWithTimeout, cancel := context.WithTimeout(ctx, d.Timeout(schema.TimeoutRead))
+	defer cancel()
+	instance, err := c.InstanceInfo(ctxWithTimeout, instanceName)
 	if err != nil {
 		if tpgresource.IsNotFoundGrpcError(err) {
 			log.Printf("[WARN] Removing %s because it's gone", instanceName)
@@ -271,7 +274,7 @@ func resourceBigtableInstanceRead(d *schema.ResourceData, meta interface{}) erro
 		return fmt.Errorf("Error setting project: %s", err)
 	}
 
-	clusters, err := c.Clusters(ctx, instance.Name)
+	clusters, err := c.Clusters(ctxWithTimeout, instance.Name)
 	if err != nil {
 		partiallyUnavailableErr, ok := err.(bigtable.ErrPartiallyUnavailable)
 

website/docs/r/bigtable_instance.html.markdown

@@ -150,6 +150,7 @@ This resource provides the following
 
 - `create` - Default is 60 minutes.
 - `update` - Default is 60 minutes.
+- `read` - Default is 60 minutes.
 
 Adding clusters to existing instances can take a long time. Consider setting a higher value to timeouts if you plan on doing that.