Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for the cost_analysis_enabled property on Aks #2042

Merged
merged 10 commits into from
Jul 23, 2024
Merged

Support for the cost_analysis_enabled property on Aks #2042

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
2f55677
Merge branch 'aztfmod:main' into main
kevindelmont Jun 17, 2024
955821d
Apply terraform fmt
actions-user Jul 2, 2024
91ab75a
Merge branch 'main' of https://github.com/visiativ-agora/terraform-az…
kevindelmont Jul 8, 2024
79b2fdb
add public_network_access_enabled for azurerm_postgresql_flexible_ser…
kevindelmont Jul 8, 2024
9b057fa
Revert "add public_network_access_enabled for azurerm_postgresql_flex…
kevindelmont Jul 8, 2024
694311c
Merge branch 'aztfmod:main' into main
kevindelmont Jul 23, 2024
76884b4
add cost_analysis_enabled
kevindelmont Jul 23, 2024
ae7a35d
Updated example
kevindelmont Jul 23, 2024
72a7d9b
fixed indentation
kevindelmont Jul 23, 2024
cb20dc8
fix
kevindelmont Jul 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions examples/compute/kubernetes_services/101-single-cluster/aks.tfvars
View file
Open in desktop
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
View file
Open in desktop
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
Loading