fix tests

docs updates to zonal

update cb

Author: bharathkkb

.gitignore

@@ -51,6 +51,6 @@ credentials.json
 # File to populate env vars used by Docker test runs
 .envrc
 
-# ignore generated ASM yamls in /workspace/test/fixtures/simple_regional_with_asm as it is a test
+# ignore generated ASM yamls in /workspace/test/fixtures/simple_zonal_with_asm as it is a test
 # in a production scenario these files are expected to be checked in
-/test/fixtures/simple_regional_with_asm/asm-dir
+/test/fixtures/simple_zonal_with_asm/asm-dir

.kitchen.yml

@@ -234,12 +234,12 @@ suites:
       systems:
         - name: safer_cluster_iap_bastion
           backend: local
-  - name: "simple_regional_with_asm"
+  - name: "simple_zonal_with_asm"
     driver:
-      root_module_directory: test/fixtures/simple_regional_with_asm
+      root_module_directory: test/fixtures/simple_zonal_with_asm
     verifier:
       systems:
-        - name: simple_regional_with_asm
+        - name: simple_zonal_with_asm
           backend: local
           controls:
             - gcloud

build/int.cloudbuild.yaml

@@ -404,26 +404,26 @@ steps:
     - verify safer-cluster-iap-bastion-local
   name: 'gcr.io/cloud-foundation-cicd/$_DOCKER_IMAGE_DEVELOPER_TOOLS:$_DOCKER_TAG_VERSION_DEVELOPER_TOOLS'
   args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do destroy safer-cluster-iap-bastion-local']
-- id: create simple-regional-with-asm-local
+- id: create simple-zonal-with-asm-local
   waitFor:
     - prepare
   name: 'gcr.io/cloud-foundation-cicd/$_DOCKER_IMAGE_DEVELOPER_TOOLS:$_DOCKER_TAG_VERSION_DEVELOPER_TOOLS'
-  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do create simple-regional-with-asm-local']
-- id: converge simple-regional-with-asm-local
+  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do create simple-zonal-with-asm-local']
+- id: converge simple-zonal-with-asm-local
   waitFor:
-    - create simple-regional-with-asm-local
+    - create simple-zonal-with-asm-local
   name: 'gcr.io/cloud-foundation-cicd/$_DOCKER_IMAGE_DEVELOPER_TOOLS:$_DOCKER_TAG_VERSION_DEVELOPER_TOOLS'
-  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do converge simple-regional-with-asm-local']
-- id: verify simple-regional-with-asm-local
+  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do converge simple-zonal-with-asm-local']
+- id: verify simple-zonal-with-asm-local
   waitFor:
-    - converge simple-regional-with-asm-local
+    - converge simple-zonal-with-asm-local
   name: 'gcr.io/cloud-foundation-cicd/$_DOCKER_IMAGE_DEVELOPER_TOOLS:$_DOCKER_TAG_VERSION_DEVELOPER_TOOLS'
-  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do verify simple-regional-with-asm-local']
-- id: destroy simple-regional-with-asm-local
+  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do verify simple-zonal-with-asm-local']
+- id: destroy simple-zonal-with-asm-local
   waitFor:
-    - verify simple-regional-with-asm-local
+    - verify simple-zonal-with-asm-local
   name: 'gcr.io/cloud-foundation-cicd/$_DOCKER_IMAGE_DEVELOPER_TOOLS:$_DOCKER_TAG_VERSION_DEVELOPER_TOOLS'
-  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do destroy simple-regional-with-asm-local']
+  args: ['/bin/bash', '-c', 'source /usr/local/bin/task_helper_functions.sh && kitchen_do destroy simple-zonal-with-asm-local']
 tags:
 - 'ci'
 - 'integration'

examples/simple_regional_with_asm/README.md renamed to examples/simple_zonal_with_asm/README.md

@@ -1,6 +1,6 @@
-# Simple Regional Cluster with ASM
+# Simple Zonal Cluster with ASM
 
-This example illustrates how to create a simple regional cluster with ASM.
+This example illustrates how to create a simple zonal cluster with ASM.
 
 <!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
 ## Inputs
@@ -14,6 +14,7 @@ This example illustrates how to create a simple regional cluster with ASM.
 | project\_id | The project ID to host the cluster in | string | n/a | yes |
 | region | The region to host the cluster in | string | n/a | yes |
 | subnetwork | The subnetwork to host the cluster in | string | n/a | yes |
+| zones | The zone to host the cluster in (required if is a zonal cluster) | list(string) | n/a | yes |
 
 ## Outputs
 

examples/simple_regional_with_asm/main.tf renamed to examples/simple_zonal_with_asm/main.tf

@@ -15,7 +15,7 @@
  */
 
 locals {
-  cluster_type = "simple-regional-asm2"
+  cluster_type = "simple-zonal-asm"
 }
 
 provider "google-beta" {
@@ -31,9 +31,10 @@ module "gke" {
   source                  = "../../modules/beta-public-cluster/"
   project_id              = var.project_id
   name                    = "${local.cluster_type}-cluster${var.cluster_name_suffix}"
-  regional                = true
-  release_channel         = "REGULAR"
+  regional                = false
   region                  = var.region
+  zones                   = var.zones
+  release_channel         = "REGULAR"
   network                 = var.network
   subnetwork              = var.subnetwork
   ip_range_pods           = var.ip_range_pods
@@ -42,11 +43,11 @@ module "gke" {
   cluster_resource_labels = { "mesh_id" : "proj-${data.google_project.project.number}" }
   node_pools = [
     {
-      name        = "asm-node-pool"
-      autoscaling = false
+      name         = "asm-node-pool"
+      autoscaling  = false
+      auto_upgrade = true
       # ASM requires minimum 4 nodes and e2-standard-4
-      # As this is a regional cluster we have node_count * 3 = 6 nodes
-      node_count   = 2
+      node_count   = 4
       machine_type = "e2-standard-4"
     },
   ]

examples/simple_regional_with_asm/outputs.tf renamed to examples/simple_zonal_with_asm/outputs.tf

@@ -27,6 +27,11 @@ variable "region" {
   description = "The region to host the cluster in"
 }
 
+variable "zones" {
+  type        = list(string)
+  description = "The zone to host the cluster in (required if is a zonal cluster)"
+}
+
 variable "network" {
   description = "The VPC network to host the cluster in"
 }

examples/simple_regional_with_asm/test_outputs.tf renamed to examples/simple_zonal_with_asm/test_outputs.tf

@@ -9,7 +9,7 @@ Specifically, this module automates the following steps for [installing ASM](htt
 
 ## Usage
 
-There is a [full example](../../examples/simple_regional_with_asm) provided. Simple usage is as follows:
+There is a [full example](../../examples/simple_zonal_with_asm) provided. Simple usage is as follows:
 
 ```tf
 module "asm" {

examples/simple_regional_with_asm/variables.tf renamed to examples/simple_zonal_with_asm/variables.tf

@@ -19,11 +19,12 @@ data "google_project" "project" {
 }
 
 module "example" {
-  source = "../../../examples/simple_regional_with_asm"
+  source = "../../../examples/simple_zonal_with_asm"
 
   project_id          = var.project_ids[2]
   cluster_name_suffix = "-${random_string.suffix.result}"
   region              = var.region
+  zones               = slice(var.zones, 0, 1)
   network             = google_compute_network.main.name
   subnetwork          = google_compute_subnetwork.main.name
   ip_range_pods       = google_compute_subnetwork.main.secondary_ip_range[0].range_name

examples/simple_regional_with_asm/versions.tf renamed to examples/simple_zonal_with_asm/versions.tf

@@ -64,6 +64,7 @@
             "disabled" => true,
           },
           "networkPolicyConfig" => {},
+          "dnsCacheConfig" => {},
         })
       end
     end

modules/asm/README.md

@@ -56,6 +56,7 @@
             "disabled" => true,
           },
           "networkPolicyConfig" => {},
+          "dnsCacheConfig" => {},
         })
       end
     end

test/fixtures/simple_regional_with_asm/example.tf renamed to test/fixtures/simple_zonal_with_asm/example.tf

@@ -1,4 +1,4 @@
-name: simple_regional_with_asm
+name: simple_zonal_with_asm
 attributes:
   - name: cluster_name
     required: true

test/fixtures/simple_regional_with_asm/network.tf renamed to test/fixtures/simple_zonal_with_asm/network.tf

@@ -50,6 +50,7 @@
             "disabled" => true,
           },
           "networkPolicyConfig" => {},
+          "dnsCacheConfig" => {},
         })
       end
     end

test/fixtures/simple_regional_with_asm/outputs.tf renamed to test/fixtures/simple_zonal_with_asm/outputs.tf

@@ -14,11 +14,15 @@
  * limitations under the License.
  */
 
+resource "random_id" "random_project_id_suffix" {
+  byte_length = 4
+}
+
 module "gke-project-1" {
   source  = "terraform-google-modules/project-factory/google"
   version = "~> 8.0"
 
-  name                 = "ci-gke"
+  name                 = "ci-gke-${random_id.random_project_id_suffix.hex}"
   random_project_id    = true
   org_id               = var.org_id
   folder_id            = var.folder_id
@@ -47,7 +51,7 @@ module "gke-project-2" {
   source  = "terraform-google-modules/project-factory/google"
   version = "~> 8.0"
 
-  name                 = "ci-gke"
+  name                 = "ci-gke-${random_id.random_project_id_suffix.hex}"
   random_project_id    = true
   org_id               = var.org_id
   folder_id            = var.folder_id
@@ -73,13 +77,14 @@ module "gke-project-2" {
 # apis as documented https://cloud.google.com/service-mesh/docs/gke-install-new-cluster#setting_up_your_project
 module "gke-project-asm" {
   source  = "terraform-google-modules/project-factory/google"
-  version = "~> 3.0"
+  version = "~> 8.0"
 
-  name              = "ci-gke-asm"
-  random_project_id = true
-  org_id            = var.org_id
-  folder_id         = var.folder_id
-  billing_account   = var.billing_account
+  name                 = "ci-gke-asm-${random_id.random_project_id_suffix.hex}"
+  random_project_id    = true
+  org_id               = var.org_id
+  folder_id            = var.folder_id
+  billing_account      = var.billing_account
+  skip_gcloud_download = true
 
   activate_apis = [
     "container.googleapis.com",