Skip to content

Commit

Permalink
Merge pull request #2067 from caiovbraga/fix-aks-default-nodepool-upg…
Browse files Browse the repository at this point in the history
…rade-settings

AKS: Fix default nodepool upgrade settings
  • Loading branch information
arnaudlh authored Nov 11, 2024
2 parents 0cdb3ba + ec51ee3 commit 55946c7
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion modules/compute/aks/aks.tf
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,9 @@ resource "azurerm_kubernetes_cluster" "aks" {
dynamic "upgrade_settings" {
for_each = try(var.settings.default_node_pool.upgrade_settings, null) == null ? [] : [1]
content {
max_surge = upgrade_settings.value.max_surge
drain_timeout_in_minutes = try(var.settings.default_node_pool.upgrade_settings.drain_timeout_in_minutes, null)
node_soak_duration_in_minutes = try(var.settings.default_node_pool.upgrade_settings.node_soak_duration_in_minutes, null)
max_surge = var.settings.default_node_pool.upgrade_settings.max_surge
}
}

Expand Down

0 comments on commit 55946c7

Please sign in to comment.