Skip to content

Commit

Permalink
Revert "Re-use the metadata_labels variable for node_labels, just as …
Browse files Browse the repository at this point in the history
…in eks"

This reverts commit 170bf26, because adding
the node_labels to the default node pool requires a destroy and re-create on
the AKS cluster.

This is likely due to the node_pool using the legacy `default_node_pool`
block of the `azurerm_kubernetes_cluster` resource. Must migrate to using
a `azurerm_kubernetes_cluster_node_pool`. Will tackle this as part of
revamping node pools support in general.

Opening an issue to track this:

#172
  • Loading branch information
pst committed Apr 6, 2021
1 parent 44e73dc commit c723a7d
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion azurerm/_modules/aks/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ resource "azurerm_kubernetes_cluster" "current" {

vnet_subnet_id = var.network_plugin == "azure" ? azurerm_subnet.current[0].id : null
max_pods = var.max_pods
node_labels = var.metadata_labels
}

network_profile {
Expand Down

0 comments on commit c723a7d

Please sign in to comment.