Support for the cost_analysis_enabled property on Aks (#2042)

* Apply terraform fmt * add public_network_access_enabled for azurerm_postgresql_flexible_server (#21) add new argument public_network_access_enabled = try(var.settings.public_network_access_enabled, null) * Revert "add public_network_access_enabled for azurerm_postgresql_flexible_ser…" (#25) This reverts commit 79b2fdb. * add cost_analysis_enabled * Updated example * fixed indentation * fix --------- Co-authored-by: GitHub Action <[email protected]>
aztfmod · Jul 23, 2024 · 909377e · 909377e
1 parent fb32f2a
commit 909377e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/examples/compute/kubernetes_services/101-single-cluster/aks.tfvars b/examples/compute/kubernetes_services/101-single-cluster/aks.tfvars
@@ -18,6 +18,8 @@ aks_clusters = {
     resource_group_key = "aks_re1"
     os_type            = "Linux"
 
+    cost_analysis_enabled = true
+
     identity = {
       type = "SystemAssigned"
     }

diff --git a/modules/compute/aks/aks.tf b/modules/compute/aks/aks.tf
@@ -161,7 +161,7 @@ resource "azurerm_kubernetes_cluster" "aks" {
       subnet_name = aci_connector_linux.value.subnet_name
     }
   }
-
+  cost_analysis_enabled            = try(var.settings.cost_analysis_enabled, null)
   azure_policy_enabled             = can(var.settings.addon_profile.azure_policy) || can(var.settings.azure_policy_enabled) == false ? try(var.settings.addon_profile.azure_policy.0.enabled, null) : var.settings.azure_policy_enabled
   http_application_routing_enabled = can(var.settings.addon_profile.http_application_routing) || can(var.settings.http_application_routing_enabled) == false ? try(var.settings.addon_profile.http_application_routing.0.enabled, null) : var.settings.http_application_routing_enabled