Skip to content

Commit

Permalink
Merge pull request #3373 from consideRatio/pr/optional-gcp-node_version
Browse files Browse the repository at this point in the history
terraform, gcp: enable specification of individual node pools version
  • Loading branch information
consideRatio authored Nov 6, 2023
2 parents 881921c + ca7a62e commit a236bc6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion terraform/gcp/cluster.tf
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ resource "google_container_node_pool" "notebook" {
cluster = google_container_cluster.cluster.name
project = google_container_cluster.cluster.project
location = google_container_cluster.cluster.location
version = var.k8s_versions.notebook_nodes_version
version = try(each.value.node_version, var.k8s_versions.notebook_nodes_version)

# terraform treats null same as unset, so we only set the node_locations
# here if it is explicitly overriden. If not, it will just inherit whatever
Expand Down
3 changes: 2 additions & 1 deletion terraform/gcp/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,8 @@ variable "notebook_nodes" {
{}
),
resource_labels : optional(map(string), {}),
zones : optional(list(string), [])
zones : optional(list(string), []),
node_version : optional(string, ""),
}))
description = "Notebook node pools to create"
default = {}
Expand Down

0 comments on commit a236bc6

Please sign in to comment.