From 55118379582999df84c4242f5ca1c90a1ece6989 Mon Sep 17 00:00:00 2001
From: Modular Magician <magic-modules@google.com>
Date: Tue, 17 Mar 2020 17:59:36 +0000
Subject: [PATCH] GA ILB global access (#3253)

Signed-off-by: Modular Magician <magic-modules@google.com>
---
 .changelog/3253.txt                                  |  3 +++
 ...esource_compute_forwarding_rule_generated_test.go | 12 ++++++------
 website/docs/r/compute_forwarding_rule.html.markdown |  7 +------
 3 files changed, 10 insertions(+), 12 deletions(-)
 create mode 100644 .changelog/3253.txt

diff --git a/.changelog/3253.txt b/.changelog/3253.txt
new file mode 100644
index 0000000000..928e37521f
--- /dev/null
+++ b/.changelog/3253.txt
@@ -0,0 +1,3 @@
+```release-note:enhancement
+compute: Added `allow_global_access` for to `google_compute_forwarding_rule` resource.
+```
diff --git a/google-beta/resource_compute_forwarding_rule_generated_test.go b/google-beta/resource_compute_forwarding_rule_generated_test.go
index 3c5b7b6757..3601ea8551 100644
--- a/google-beta/resource_compute_forwarding_rule_generated_test.go
+++ b/google-beta/resource_compute_forwarding_rule_generated_test.go
@@ -33,12 +33,17 @@ func TestAccComputeForwardingRule_forwardingRuleGlobalInternallbExample(t *testi
 
 	resource.Test(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t) },
-		Providers:    testAccProvidersOiCS,
+		Providers:    testAccProviders,
 		CheckDestroy: testAccCheckComputeForwardingRuleDestroy,
 		Steps: []resource.TestStep{
 			{
 				Config: testAccComputeForwardingRule_forwardingRuleGlobalInternallbExample(context),
 			},
+			{
+				ResourceName:      "google_compute_forwarding_rule.default",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 		},
 	})
 }
@@ -47,7 +52,6 @@ func testAccComputeForwardingRule_forwardingRuleGlobalInternallbExample(context
 	return Nprintf(`
 // Forwarding rule for Internal Load Balancing
 resource "google_compute_forwarding_rule" "default" {
-  provider = "google-beta"
   name                  = "tf-test-website-forwarding-rule%{random_suffix}"
   region                = "us-central1"
   load_balancing_scheme = "INTERNAL"
@@ -58,13 +62,11 @@ resource "google_compute_forwarding_rule" "default" {
   subnetwork            = "${google_compute_subnetwork.default.name}"
 }
 resource "google_compute_region_backend_service" "backend" {
-  provider = "google-beta"
   name                  = "tf-test-website-backend%{random_suffix}"
   region                = "us-central1"
   health_checks         = ["${google_compute_health_check.hc.self_link}"]
 }
 resource "google_compute_health_check" "hc" {
-  provider = "google-beta"
   name               = "check-tf-test-website-backend%{random_suffix}"
   check_interval_sec = 1
   timeout_sec        = 1
@@ -73,12 +75,10 @@ resource "google_compute_health_check" "hc" {
   }
 }
 resource "google_compute_network" "default" {
-  provider = "google-beta"
   name = "tf-test-website-net%{random_suffix}"
   auto_create_subnetworks = false
 }
 resource "google_compute_subnetwork" "default" {
-  provider = "google-beta"
   name          = "tf-test-website-net%{random_suffix}"
   ip_cidr_range = "10.0.0.0/16"
   region        = "us-central1"
diff --git a/website/docs/r/compute_forwarding_rule.html.markdown b/website/docs/r/compute_forwarding_rule.html.markdown
index 17d39dd757..5090660648 100644
--- a/website/docs/r/compute_forwarding_rule.html.markdown
+++ b/website/docs/r/compute_forwarding_rule.html.markdown
@@ -44,7 +44,6 @@ To get more information about ForwardingRule, see:
 ```hcl
 // Forwarding rule for Internal Load Balancing
 resource "google_compute_forwarding_rule" "default" {
-  provider = "google-beta"
   name                  = "website-forwarding-rule"
   region                = "us-central1"
   load_balancing_scheme = "INTERNAL"
@@ -55,13 +54,11 @@ resource "google_compute_forwarding_rule" "default" {
   subnetwork            = "${google_compute_subnetwork.default.name}"
 }
 resource "google_compute_region_backend_service" "backend" {
-  provider = "google-beta"
   name                  = "website-backend"
   region                = "us-central1"
   health_checks         = ["${google_compute_health_check.hc.self_link}"]
 }
 resource "google_compute_health_check" "hc" {
-  provider = "google-beta"
   name               = "check-website-backend"
   check_interval_sec = 1
   timeout_sec        = 1
@@ -70,12 +67,10 @@ resource "google_compute_health_check" "hc" {
   }
 }
 resource "google_compute_network" "default" {
-  provider = "google-beta"
   name = "website-net"
   auto_create_subnetworks = false
 }
 resource "google_compute_subnetwork" "default" {
-  provider = "google-beta"
   name          = "website-net"
   ip_cidr_range = "10.0.0.0/16"
   region        = "us-central1"
@@ -480,7 +475,7 @@ The following arguments are supported:
   object.
 
 * `allow_global_access` -
-  (Optional, [Beta](https://terraform.io/docs/providers/google/guides/provider_versions.html))
+  (Optional)
   If true, clients can access ILB from all regions.
   Otherwise only allows from the local region the ILB is located at.