Merge pull request #50 from ogreface/issue_43

Issue 43: Fix issue with single zone, and update test for the problem.

Author: Jberlinsky

main.tf

@@ -51,7 +51,7 @@ locals {
   }
 
   cluster_type_output_regional_zones = "${concat(google_container_cluster.primary.*.additional_zones, list(list()))}"
-  cluster_type_output_zonal_zones    = "${concat(google_container_cluster.zonal_primary.*.additional_zones, list(list()))}"
+  cluster_type_output_zonal_zones    = "${concat(slice(var.zones,1,length(var.zones)), list(list()))}"
 
   cluster_type_output_zones = {
     regional = "${local.cluster_type_output_regional_zones[0]}"

test/fixtures/simple_zonal/example.tf

@@ -21,7 +21,7 @@ module "example" {
   credentials_path               = "${local.credentials_path}"
   cluster_name_suffix            = "-${random_string.suffix.result}"
   region                         = "${var.region}"
-  zones                          = ["${var.zones}"]
+  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}"

test/integration/simple_zonal/controls/gcloud.rb

@@ -42,6 +42,10 @@
         expect(data['location']).to match(/^(.*)[1-9]-[a-z]$/)
       end
 
+      it "is single zoned" do
+        expect(data['locations'].size).to eq 1
+      end
+
       it "has the expected initial cluster version" do
         expect(data['initialClusterVersion']).to eq "1.11.4-gke.13"
       end

variables.tf

@@ -39,7 +39,7 @@ variable "region" {
 variable "zones" {
   type        = "list"
   description = "The zones to host the cluster in (optional if regional cluster / required if zonal)"
-  default     = []
+  default     = [""]
 }
 
 variable "network" {