Modify test to use lowercase, add case diff suppress on node config i…

…mage_type (#4029) (#2538) Signed-off-by: Modular Magician <[email protected]>
hashicorp · Oct 2, 2020 · 3e71bf8 · 3e71bf8
1 parent 0eb8fff
commit 3e71bf8
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/.changelog/4029.txt b/.changelog/4029.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+container: fixed `node_config.image_type` perma-diff when specified in lower case.
+```
diff --git a/google-beta/node_config.go b/google-beta/node_config.go
@@ -69,9 +69,10 @@ func schemaNodeConfig() *schema.Schema {
 				},
 
 				"image_type": {
-					Type:     schema.TypeString,
-					Optional: true,
-					Computed: true,
+					Type:             schema.TypeString,
+					Optional:         true,
+					Computed:         true,
+					DiffSuppressFunc: caseDiffSuppress,
 				},
 
 				"labels": {

diff --git a/google-beta/resource_container_cluster_test.go b/google-beta/resource_container_cluster_test.go
@@ -2830,7 +2830,7 @@ resource "google_container_cluster" "with_node_config" {
     }
 
     // Updatable fields
-    image_type = "COS"
+    image_type = "cos"
   }
 }
 `, clusterName)