Skip to content

Commit 419078d

Browse files
psiegmannyardiapeabody
psiegmannyardi
and
apeabody
authored
fix: Node Pool enable_gcfs true:false (#1976)
Co-authored-by: Andrew Peabody <[email protected]>
1 parent a083437 commit 419078d

File tree

10 files changed

+25
-24
lines changed

10 files changed

+25
-24
lines changed

‎autogen/main/cluster.tf.tmpl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -516,7 +516,7 @@ resource "google_container_cluster" "primary" {
516516
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
517517
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
518518
dynamic "gcfs_config" {
519-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
519+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
520520
content {
521521
enabled = gcfs_config.value
522522
}
@@ -887,7 +887,7 @@ resource "google_container_node_pool" "windows_pools" {
887887
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
888888
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
889889
dynamic "gcfs_config" {
890-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
890+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
891891
content {
892892
enabled = gcfs_config.value
893893
}

‎cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ resource "google_container_cluster" "primary" {
393393
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
394394
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
395395
dynamic "gcfs_config" {
396-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
396+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
397397
content {
398398
enabled = gcfs_config.value
399399
}
@@ -593,7 +593,7 @@ resource "google_container_node_pool" "pools" {
593593
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
594594
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
595595
dynamic "gcfs_config" {
596-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
596+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
597597
content {
598598
enabled = gcfs_config.value
599599
}
@@ -875,7 +875,7 @@ resource "google_container_node_pool" "windows_pools" {
875875
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
876876
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
877877
dynamic "gcfs_config" {
878-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
878+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
879879
content {
880880
enabled = gcfs_config.value
881881
}

‎examples/simple_regional_with_ipv6/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ provider "kubernetes" {
2727
}
2828

2929
module "gke" {
30-
source = "../../"
30+
source = "terraform-google-modules/kubernetes-engine/google"
3131
project_id = var.project_id
3232
name = "${local.cluster_type}-cluster${var.cluster_name_suffix}"
3333
regional = true

‎modules/beta-private-cluster-update-variant/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ resource "google_container_cluster" "primary" {
439439
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
440440
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
441441
dynamic "gcfs_config" {
442-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
442+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
443443
content {
444444
enabled = gcfs_config.value
445445
}
@@ -760,7 +760,7 @@ resource "google_container_node_pool" "pools" {
760760
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
761761
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
762762
dynamic "gcfs_config" {
763-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
763+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
764764
content {
765765
enabled = gcfs_config.value
766766
}
@@ -1056,7 +1056,7 @@ resource "google_container_node_pool" "windows_pools" {
10561056
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
10571057
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
10581058
dynamic "gcfs_config" {
1059-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
1059+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
10601060
content {
10611061
enabled = gcfs_config.value
10621062
}

‎modules/beta-private-cluster/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ resource "google_container_cluster" "primary" {
439439
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
440440
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
441441
dynamic "gcfs_config" {
442-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
442+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
443443
content {
444444
enabled = gcfs_config.value
445445
}
@@ -678,7 +678,7 @@ resource "google_container_node_pool" "pools" {
678678
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
679679
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
680680
dynamic "gcfs_config" {
681-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
681+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
682682
content {
683683
enabled = gcfs_config.value
684684
}
@@ -973,7 +973,7 @@ resource "google_container_node_pool" "windows_pools" {
973973
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
974974
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
975975
dynamic "gcfs_config" {
976-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
976+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
977977
content {
978978
enabled = gcfs_config.value
979979
}

‎modules/beta-public-cluster-update-variant/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ resource "google_container_cluster" "primary" {
439439
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
440440
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
441441
dynamic "gcfs_config" {
442-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
442+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
443443
content {
444444
enabled = gcfs_config.value
445445
}
@@ -739,7 +739,7 @@ resource "google_container_node_pool" "pools" {
739739
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
740740
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
741741
dynamic "gcfs_config" {
742-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
742+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
743743
content {
744744
enabled = gcfs_config.value
745745
}
@@ -1035,7 +1035,7 @@ resource "google_container_node_pool" "windows_pools" {
10351035
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
10361036
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
10371037
dynamic "gcfs_config" {
1038-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
1038+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
10391039
content {
10401040
enabled = gcfs_config.value
10411041
}

‎modules/beta-public-cluster/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,7 @@ resource "google_container_cluster" "primary" {
439439
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
440440
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
441441
dynamic "gcfs_config" {
442-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
442+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
443443
content {
444444
enabled = gcfs_config.value
445445
}
@@ -657,7 +657,7 @@ resource "google_container_node_pool" "pools" {
657657
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
658658
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
659659
dynamic "gcfs_config" {
660-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
660+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
661661
content {
662662
enabled = gcfs_config.value
663663
}
@@ -952,7 +952,7 @@ resource "google_container_node_pool" "windows_pools" {
952952
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
953953
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
954954
dynamic "gcfs_config" {
955-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
955+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
956956
content {
957957
enabled = gcfs_config.value
958958
}

‎modules/private-cluster-update-variant/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ resource "google_container_cluster" "primary" {
393393
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
394394
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
395395
dynamic "gcfs_config" {
396-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
396+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
397397
content {
398398
enabled = gcfs_config.value
399399
}
@@ -696,7 +696,7 @@ resource "google_container_node_pool" "pools" {
696696
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
697697
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
698698
dynamic "gcfs_config" {
699-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
699+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
700700
content {
701701
enabled = gcfs_config.value
702702
}
@@ -979,7 +979,7 @@ resource "google_container_node_pool" "windows_pools" {
979979
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
980980
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
981981
dynamic "gcfs_config" {
982-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
982+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
983983
content {
984984
enabled = gcfs_config.value
985985
}

‎modules/private-cluster/cluster.tf

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ resource "google_container_cluster" "primary" {
393393
min_cpu_platform = lookup(var.node_pools[0], "min_cpu_platform", "")
394394
enable_confidential_storage = lookup(var.node_pools[0], "enable_confidential_storage", false)
395395
dynamic "gcfs_config" {
396-
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : []
396+
for_each = lookup(var.node_pools[0], "enable_gcfs", false) ? [true] : [false]
397397
content {
398398
enabled = gcfs_config.value
399399
}
@@ -614,7 +614,7 @@ resource "google_container_node_pool" "pools" {
614614
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
615615
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
616616
dynamic "gcfs_config" {
617-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
617+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
618618
content {
619619
enabled = gcfs_config.value
620620
}
@@ -896,7 +896,7 @@ resource "google_container_node_pool" "windows_pools" {
896896
min_cpu_platform = lookup(each.value, "min_cpu_platform", "")
897897
enable_confidential_storage = lookup(each.value, "enable_confidential_storage", false)
898898
dynamic "gcfs_config" {
899-
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : []
899+
for_each = lookup(each.value, "enable_gcfs", false) ? [true] : [false]
900900
content {
901901
enabled = gcfs_config.value
902902
}

‎test/integration/private_zonal_with_networking/testdata/TestPrivateZonalWithNetworking.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -210,6 +210,7 @@
210210
"config": {
211211
"diskSizeGb": 100,
212212
"diskType": "pd-standard",
213+
"gcfsConfig": {},
213214
"imageType": "COS_CONTAINERD",
214215
"labels": {
215216
"cluster_name": "CLUSTER_NAME",

0 commit comments

Comments
 (0)