Skip to content

Commit b006593

Browse files
authored
feat!: update variant - recreate node pools on max_pods_per_node or pod_range change (#1464)
* fix: add missing max_pods_per_node & pod_range keepers for update variant module * max_pods_per_node & pod_range keepers autogen
1 parent 4278f2c commit b006593

File tree

4 files changed

+20
-8
lines changed

4 files changed

+20
-8
lines changed

autogen/main/cluster.tf.tmpl

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -497,7 +497,9 @@ locals {
497497
"enable_integrity_monitoring",
498498
"local_ssd_count",
499499
"machine_type",
500+
"max_pods_per_node",
500501
"min_cpu_platform",
502+
"pod_range",
501503
"preemptible",
502504
"spot",
503505
"service_account",
@@ -508,8 +510,9 @@ locals {
508510
}
509511

510512
# This keepers list is based on the terraform google provider schemaNodeConfig
511-
# resources where "ForceNew" is "true". schemaNodeConfig can be found in node_config.go at
512-
# https://github.com/terraform-providers/terraform-provider-google/blob/master/google/node_config.go#L22
513+
# resources where "ForceNew" is "true". schemaNodeConfig can be found in resource_container_node_pool.go at
514+
# https://github.com/hashicorp/terraform-provider-google/blob/main/google/resource_container_node_pool.go and node_config.go at
515+
# https://github.com/terraform-providers/terraform-provider-google/blob/main/google/node_config.go
513516
resource "random_id" "name" {
514517
for_each = merge(local.node_pools, local.windows_node_pools)
515518
byte_length = 2

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,9 @@ locals {
437437
"enable_integrity_monitoring",
438438
"local_ssd_count",
439439
"machine_type",
440+
"max_pods_per_node",
440441
"min_cpu_platform",
442+
"pod_range",
441443
"preemptible",
442444
"spot",
443445
"service_account",
@@ -448,8 +450,9 @@ locals {
448450
}
449451

450452
# This keepers list is based on the terraform google provider schemaNodeConfig
451-
# resources where "ForceNew" is "true". schemaNodeConfig can be found in node_config.go at
452-
# https://github.com/terraform-providers/terraform-provider-google/blob/master/google/node_config.go#L22
453+
# resources where "ForceNew" is "true". schemaNodeConfig can be found in resource_container_node_pool.go at
454+
# https://github.com/hashicorp/terraform-provider-google/blob/main/google/resource_container_node_pool.go and node_config.go at
455+
# https://github.com/terraform-providers/terraform-provider-google/blob/main/google/node_config.go
453456
resource "random_id" "name" {
454457
for_each = merge(local.node_pools, local.windows_node_pools)
455458
byte_length = 2

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,9 @@ locals {
418418
"enable_integrity_monitoring",
419419
"local_ssd_count",
420420
"machine_type",
421+
"max_pods_per_node",
421422
"min_cpu_platform",
423+
"pod_range",
422424
"preemptible",
423425
"spot",
424426
"service_account",
@@ -429,8 +431,9 @@ locals {
429431
}
430432

431433
# This keepers list is based on the terraform google provider schemaNodeConfig
432-
# resources where "ForceNew" is "true". schemaNodeConfig can be found in node_config.go at
433-
# https://github.com/terraform-providers/terraform-provider-google/blob/master/google/node_config.go#L22
434+
# resources where "ForceNew" is "true". schemaNodeConfig can be found in resource_container_node_pool.go at
435+
# https://github.com/hashicorp/terraform-provider-google/blob/main/google/resource_container_node_pool.go and node_config.go at
436+
# https://github.com/terraform-providers/terraform-provider-google/blob/main/google/node_config.go
434437
resource "random_id" "name" {
435438
for_each = merge(local.node_pools, local.windows_node_pools)
436439
byte_length = 2

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,9 @@ locals {
329329
"enable_integrity_monitoring",
330330
"local_ssd_count",
331331
"machine_type",
332+
"max_pods_per_node",
332333
"min_cpu_platform",
334+
"pod_range",
333335
"preemptible",
334336
"spot",
335337
"service_account",
@@ -340,8 +342,9 @@ locals {
340342
}
341343

342344
# This keepers list is based on the terraform google provider schemaNodeConfig
343-
# resources where "ForceNew" is "true". schemaNodeConfig can be found in node_config.go at
344-
# https://github.com/terraform-providers/terraform-provider-google/blob/master/google/node_config.go#L22
345+
# resources where "ForceNew" is "true". schemaNodeConfig can be found in resource_container_node_pool.go at
346+
# https://github.com/hashicorp/terraform-provider-google/blob/main/google/resource_container_node_pool.go and node_config.go at
347+
# https://github.com/terraform-providers/terraform-provider-google/blob/main/google/node_config.go
345348
resource "random_id" "name" {
346349
for_each = merge(local.node_pools, local.windows_node_pools)
347350
byte_length = 2

0 commit comments

Comments
 (0)