Skip to content

Commit 6239588

Browse files
committed
change to null
Signed-off-by: drfaust92 <[email protected]>
1 parent 96fa379 commit 6239588

File tree

20 files changed

+32
-32
lines changed

20 files changed

+32
-32
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ Then perform the following commands on the root folder:
162162
| deletion\_protection | Whether or not to allow Terraform to destroy the cluster. | `bool` | `true` | no |
163163
| description | The description of the cluster | `string` | `""` | no |
164164
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
165-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
165+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
166166
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
167167
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
168168
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

autogen/main/variables.tf.tmpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -983,7 +983,7 @@ variable "enable_l4_ilb_subsetting" {
983983
variable "disable_l4_lb_firewall_reconciliation" {
984984
type = bool
985985
description = "Disable L4 Load Balancer firewall reconciliation"
986-
default = false
986+
default = null
987987
}
988988
{% if beta_cluster %}
989989
{% if autopilot_cluster != true %}

examples/simple_autopilot_private/main.tf

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -37,19 +37,19 @@ module "gke" {
3737
source = "terraform-google-modules/kubernetes-engine/google//modules/beta-autopilot-private-cluster"
3838
version = "~> 36.0"
3939

40-
project_id = var.project_id
41-
name = "${local.cluster_type}-cluster"
42-
regional = true
43-
region = var.region
44-
network = module.gcp-network.network_name
45-
subnetwork = local.subnet_names[index(module.gcp-network.subnets_names, local.subnet_name)]
46-
ip_range_pods = local.pods_range_name
47-
ip_range_services = local.svc_range_name
48-
release_channel = "REGULAR"
49-
enable_vertical_pod_autoscaling = true
50-
enable_private_endpoint = true
51-
enable_private_nodes = true
52-
network_tags = [local.cluster_type]
40+
project_id = var.project_id
41+
name = "${local.cluster_type}-cluster"
42+
regional = true
43+
region = var.region
44+
network = module.gcp-network.network_name
45+
subnetwork = local.subnet_names[index(module.gcp-network.subnets_names, local.subnet_name)]
46+
ip_range_pods = local.pods_range_name
47+
ip_range_services = local.svc_range_name
48+
release_channel = "REGULAR"
49+
enable_vertical_pod_autoscaling = true
50+
enable_private_endpoint = true
51+
enable_private_nodes = true
52+
network_tags = [local.cluster_type]
5353
# TODO: b/413643369
5454
# node_pools_cgroup_mode = "CGROUP_MODE_V2"
5555
deletion_protection = false

modules/beta-autopilot-private-cluster/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ Then perform the following commands on the root folder:
9090
| deploy\_using\_private\_endpoint | A toggle for Terraform and kubectl to connect to the master's internal IP address during deployment. | `bool` | `false` | no |
9191
| description | The description of the cluster | `string` | `""` | no |
9292
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
93-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
93+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
9494
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
9595
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `true` | no |
9696
| enable\_binary\_authorization | Enable BinAuthZ Admission controller | `bool` | `false` | no |

modules/beta-autopilot-private-cluster/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -607,7 +607,7 @@ variable "enable_l4_ilb_subsetting" {
607607
variable "disable_l4_lb_firewall_reconciliation" {
608608
type = bool
609609
description = "Disable L4 Load Balancer firewall reconciliation"
610-
default = false
610+
default = null
611611
}
612612

613613
variable "allow_net_admin" {

modules/beta-autopilot-public-cluster/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ Then perform the following commands on the root folder:
8383
| deletion\_protection | Whether or not to allow Terraform to destroy the cluster. | `bool` | `true` | no |
8484
| description | The description of the cluster | `string` | `""` | no |
8585
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
86-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
86+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
8787
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
8888
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `true` | no |
8989
| enable\_binary\_authorization | Enable BinAuthZ Admission controller | `bool` | `false` | no |

modules/beta-autopilot-public-cluster/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -571,7 +571,7 @@ variable "enable_l4_ilb_subsetting" {
571571
variable "disable_l4_lb_firewall_reconciliation" {
572572
type = bool
573573
description = "Disable L4 Load Balancer firewall reconciliation"
574-
default = false
574+
default = null
575575
}
576576

577577
variable "allow_net_admin" {

modules/beta-private-cluster-update-variant/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ Then perform the following commands on the root folder:
197197
| deploy\_using\_private\_endpoint | A toggle for Terraform and kubectl to connect to the master's internal IP address during deployment. | `bool` | `false` | no |
198198
| description | The description of the cluster | `string` | `""` | no |
199199
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
200-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
200+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
201201
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
202202
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
203203
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -926,7 +926,7 @@ variable "enable_l4_ilb_subsetting" {
926926
variable "disable_l4_lb_firewall_reconciliation" {
927927
type = bool
928928
description = "Disable L4 Load Balancer firewall reconciliation"
929-
default = false
929+
default = null
930930
}
931931

932932
variable "istio" {

modules/beta-private-cluster/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ Then perform the following commands on the root folder:
175175
| deploy\_using\_private\_endpoint | A toggle for Terraform and kubectl to connect to the master's internal IP address during deployment. | `bool` | `false` | no |
176176
| description | The description of the cluster | `string` | `""` | no |
177177
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
178-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
178+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
179179
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
180180
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
181181
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

modules/beta-private-cluster/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -926,7 +926,7 @@ variable "enable_l4_ilb_subsetting" {
926926
variable "disable_l4_lb_firewall_reconciliation" {
927927
type = bool
928928
description = "Disable L4 Load Balancer firewall reconciliation"
929-
default = false
929+
default = null
930930
}
931931

932932
variable "istio" {

modules/beta-public-cluster-update-variant/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ Then perform the following commands on the root folder:
190190
| deletion\_protection | Whether or not to allow Terraform to destroy the cluster. | `bool` | `true` | no |
191191
| description | The description of the cluster | `string` | `""` | no |
192192
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
193-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
193+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
194194
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
195195
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
196196
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -890,7 +890,7 @@ variable "enable_l4_ilb_subsetting" {
890890
variable "disable_l4_lb_firewall_reconciliation" {
891891
type = bool
892892
description = "Disable L4 Load Balancer firewall reconciliation"
893-
default = false
893+
default = null
894894
}
895895

896896
variable "istio" {

modules/beta-public-cluster/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ Then perform the following commands on the root folder:
168168
| deletion\_protection | Whether or not to allow Terraform to destroy the cluster. | `bool` | `true` | no |
169169
| description | The description of the cluster | `string` | `""` | no |
170170
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
171-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
171+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
172172
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
173173
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
174174
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

modules/beta-public-cluster/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -890,7 +890,7 @@ variable "enable_l4_ilb_subsetting" {
890890
variable "disable_l4_lb_firewall_reconciliation" {
891891
type = bool
892892
description = "Disable L4 Load Balancer firewall reconciliation"
893-
default = false
893+
default = null
894894
}
895895

896896
variable "istio" {

modules/private-cluster-update-variant/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ Then perform the following commands on the root folder:
191191
| deploy\_using\_private\_endpoint | A toggle for Terraform and kubectl to connect to the master's internal IP address during deployment. | `bool` | `false` | no |
192192
| description | The description of the cluster | `string` | `""` | no |
193193
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
194-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
194+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
195195
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
196196
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
197197
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -903,7 +903,7 @@ variable "enable_l4_ilb_subsetting" {
903903
variable "disable_l4_lb_firewall_reconciliation" {
904904
type = bool
905905
description = "Disable L4 Load Balancer firewall reconciliation"
906-
default = false
906+
default = null
907907
}
908908

909909
variable "enable_identity_service" {

modules/private-cluster/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ Then perform the following commands on the root folder:
169169
| deploy\_using\_private\_endpoint | A toggle for Terraform and kubectl to connect to the master's internal IP address during deployment. | `bool` | `false` | no |
170170
| description | The description of the cluster | `string` | `""` | no |
171171
| disable\_default\_snat | Whether to disable the default SNAT to support the private use of public IP addresses | `bool` | `false` | no |
172-
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `false` | no |
172+
| disable\_l4\_lb\_firewall\_reconciliation | Disable L4 Load Balancer firewall reconciliation | `bool` | `null` | no |
173173
| disable\_legacy\_metadata\_endpoints | Disable the /0.1/ and /v1beta1/ metadata server endpoints on the node. Changing this value will cause all node pools to be recreated. | `bool` | `true` | no |
174174
| dns\_allow\_external\_traffic | (Optional) Controls whether external traffic is allowed over the dns endpoint. | `bool` | `null` | no |
175175
| dns\_cache | The status of the NodeLocal DNSCache addon. | `bool` | `false` | no |

modules/private-cluster/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -903,7 +903,7 @@ variable "enable_l4_ilb_subsetting" {
903903
variable "disable_l4_lb_firewall_reconciliation" {
904904
type = bool
905905
description = "Disable L4 Load Balancer firewall reconciliation"
906-
default = false
906+
default = null
907907
}
908908

909909
variable "enable_identity_service" {

variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -867,7 +867,7 @@ variable "enable_l4_ilb_subsetting" {
867867
variable "disable_l4_lb_firewall_reconciliation" {
868868
type = bool
869869
description = "Disable L4 Load Balancer firewall reconciliation"
870-
default = false
870+
default = null
871871
}
872872

873873
variable "enable_identity_service" {

0 commit comments

Comments
 (0)