Skip to content

Commit

Permalink
Support for the cost_analysis_enabled property on Aks (aztfmod#2042)
Browse files Browse the repository at this point in the history
* 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]>
  • Loading branch information
kevindelmont and actions-user authored Jul 23, 2024
1 parent fb32f2a commit 909377e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ aks_clusters = {
resource_group_key = "aks_re1"
os_type = "Linux"

cost_analysis_enabled = true

identity = {
type = "SystemAssigned"
}
Expand Down
2 changes: 1 addition & 1 deletion modules/compute/aks/aks.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit 909377e

Please sign in to comment.