remove requirement form zone field in composer config.node_config (#10324) (#5320) (#3745)

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

Author: modular-magician

.changelog/5320.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+composer: removed `config.node_config.zone` requirement on `google_composer_environment`
+```

google-beta/resource_composer_environment.go

@@ -141,7 +141,8 @@ func resourceComposerEnvironment() *schema.Resource {
 								Schema: map[string]*schema.Schema{
 									"zone": {
 										Type:             schema.TypeString,
-										Required:         true,
+										Optional:         true,
+										Computed:         true,
 										ForceNew:         true,
 										DiffSuppressFunc: compareSelfLinkOrResourceName,
 										Description:      `The Compute Engine zone in which to deploy the VMs running the Apache Airflow software, specified as the zone name or relative resource name (e.g. "projects/{project}/zones/{zone}"). Must belong to the enclosing environment's project and region. This field is supported for Cloud Composer environments in versions composer-1.*.*-airflow-*.*.*.`,
@@ -1690,9 +1691,6 @@ func expandComposerEnvironmentMachineType(v interface{}, d *schema.ResourceData,
 
 	fv, err := ParseMachineTypesFieldValue(v.(string), d, config)
 	if err != nil {
-		if requiredZone == "" {
-			return "", err
-		}
 
 		// Try to construct machine type with zone/project given in config.
 		project, err := getProject(d, config)

google-beta/resource_composer_environment_test.go

@@ -563,9 +563,10 @@ resource "google_composer_environment" "test" {
 	region = "us-central1"
 	config {
 		node_config {
-			network    = google_compute_network.test.self_link
-			subnetwork = google_compute_subnetwork.test.self_link
-			zone       = "us-central1-a"
+			network    		= google_compute_network.test.self_link
+			subnetwork 		= google_compute_subnetwork.test.self_link
+			zone       		= "us-central1-a"
+			machine_type  = "n1-standard-1"
 		}
 	}
 }
@@ -971,43 +972,50 @@ locals {
 }
 resource "google_composer_environment" "test" {
 	name   = "%s"
-	region = "europe-west3"
+	region = "us-central1"
 
 		config {
+			node_config {
+      	network    			= google_compute_network.test.self_link
+      	subnetwork 			= google_compute_subnetwork.test.self_link
+    	}
+
   		software_config {
   		  image_version = local.matching_images[0]
   		}
+
   		maintenance_window {
-  			start_time = "2019-08-01T01:00:00Z"
-  			end_time = "2019-08-01T07:00:00Z"
-  			recurrence = "FREQ=WEEKLY;BYDAY=TU,WE"
+  			start_time 	= "2019-08-01T01:00:00Z"
+  			end_time 		= "2019-08-01T07:00:00Z"
+  			recurrence 	= "FREQ=WEEKLY;BYDAY=TU,WE"
   		}
+
   		workloads_config {
   			scheduler {
-  				cpu = 1.25
-  				memory_gb = 2.5
-  				storage_gb = 5.4
-  				count = 2
+  				cpu 				= 1.25
+  				memory_gb 	= 2.5
+  				storage_gb 	= 5.4
+  				count 			= 2
   			}
   			web_server {
-  				cpu = 1.75
-  				memory_gb = 3.0
-  				storage_gb = 4.4
+  				cpu 				= 1.75
+  				memory_gb 	= 3.0
+  				storage_gb 	= 4.4
   			}
   			worker {
-  				cpu = 0.5
-  				memory_gb = 2.0
-  				storage_gb = 3.4
-  				min_count = 2
-  				max_count = 5
+  				cpu 				= 0.5
+  				memory_gb 	= 2.0
+  				storage_gb 	= 3.4
+  				min_count 	= 2
+  				max_count 	= 5
   			}
   		}
-  		environment_size = "ENVIRONMENT_SIZE_MEDIUM"
+			environment_size = "ENVIRONMENT_SIZE_MEDIUM"
   		private_environment_config {
-  			enable_private_endpoint = true
-  			cloud_composer_network_ipv4_cidr_block = "10.3.192.0/24"
-        master_ipv4_cidr_block = "172.16.194.0/23"
-        cloud_sql_ipv4_cidr_block = "10.3.224.0/20"
+  			enable_private_endpoint 								= true
+  			cloud_composer_network_ipv4_cidr_block 	= "10.3.192.0/24"
+        master_ipv4_cidr_block 									= "172.16.194.0/23"
+        cloud_sql_ipv4_cidr_block 							= "10.3.224.0/20"
   		}
   	}
 
@@ -1022,7 +1030,8 @@ resource "google_compute_subnetwork" "test" {
 	name          = "%s"
 	ip_cidr_range = "10.2.0.0/16"
 	region        = "us-central1"
-	network       = google_compute_network.test.self_link
+ 	network       = google_compute_network.test.self_link
+	private_ip_google_access = true
 }
 
 `, envName, network, subnetwork)