Skip to content

Commit

Permalink
Add default for kube_dashboard as not specifying it is causing cluste…
Browse files Browse the repository at this point in the history
…rs to be recreated. (#1438)

* add kubedash default, closes #1436
  • Loading branch information
jmspring authored Jul 8, 2020
1 parent 3a0eae3 commit bfc5418
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
6 changes: 6 additions & 0 deletions cluster/azure/aks/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,12 @@ resource "azurerm_kubernetes_cluster" "cluster" {
enabled = var.oms_agent_enabled
log_analytics_workspace_id = azurerm_log_analytics_workspace.workspace.id
}

# adding this as a patch to disable azurerm provider from redeploying due to unset
# internal "optional value". To be removed when azurerm provider is fixed.
kube_dashboard {
enabled = var.enable_kube_dashboard
}
}

# This dynamic block enables managed service identity for the cluster
Expand Down
5 changes: 5 additions & 0 deletions cluster/azure/aks/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,11 @@ variable "enable_virtual_node_addon" {
default = "false"
}

variable "enable_kube_dashboard" {
type = bool
default = true
}

variable "kubeconfig_to_disk" {
description = "This disables or enables the kube config file from being written to disk."
type = string
Expand Down

0 comments on commit bfc5418

Please sign in to comment.