Skip to content

Commit

Permalink
feat: Add keeper for enable_gcfs node pool option for update varian…
Browse files Browse the repository at this point in the history
…ts (#1218)

As changing the `gcfs_config` block currently forces a replacement of
the node pool, see:
https://github.com/hashicorp/terraform-provider-google/blob/5290a9f74d491bd3a05c32cdc60773ac6c53d4ae/google/node_config.go#L119

Fixes #1217

Co-authored-by: Bharath KKB <bharathkrishnakb@gmail.com>
  • Loading branch information
lauraseidler and bharathkkb committed Apr 15, 2022
1 parent 7553a2b commit f431756
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions autogen/main/cluster.tf.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -435,6 +435,7 @@ locals {
"spot",
{% endif %}
"service_account",
"enable_gcfs",
]
}

Expand Down
1 change: 1 addition & 0 deletions modules/beta-private-cluster-update-variant/cluster.tf
Original file line number Diff line number Diff line change
Expand Up @@ -374,6 +374,7 @@ locals {
"preemptible",
"spot",
"service_account",
"enable_gcfs",
]
}

Expand Down
1 change: 1 addition & 0 deletions modules/beta-public-cluster-update-variant/cluster.tf
Original file line number Diff line number Diff line change
Expand Up @@ -355,6 +355,7 @@ locals {
"preemptible",
"spot",
"service_account",
"enable_gcfs",
]
}

Expand Down
1 change: 1 addition & 0 deletions modules/private-cluster-update-variant/cluster.tf
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ locals {
"min_cpu_platform",
"preemptible",
"service_account",
"enable_gcfs",
]
}

Expand Down

0 comments on commit f431756

Please sign in to comment.