Skip to content

Commit

Permalink
Merge pull request #4 from EnergiMidt/fix/remove-retention-policy
Browse files Browse the repository at this point in the history
fix: fjernet retention policy
  • Loading branch information
cx-lmp authored Dec 13, 2023
2 parents e4ac74e + 73ba421 commit c9d8d0e
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,6 @@ resource "azurerm_monitor_diagnostic_setting" "diagnostic_setting" {
for_each = var.log_category_types
content {
category = enabled_log.value

# TODO: The `retention_policy` has been deprecated in favor of `azurerm_storage_management_policy` resource.
# Learn more information on the deprecation [in the Azure documentation](https://aka.ms/diagnostic_settings_log_retention).
# https://github.com/hashicorp/terraform-provider-azurerm/pull/23260
# https://github.com/hashicorp/terraform-provider-azurerm/issues/23051
retention_policy {
enabled = true
}
}
}

Expand All @@ -38,14 +30,6 @@ resource "azurerm_monitor_diagnostic_setting" "diagnostic_setting" {
content {
category = metric.value
enabled = true

# TODO: The `retention_policy` has been deprecated in favor of `azurerm_storage_management_policy` resource.
# Learn more information on the deprecation [in the Azure documentation](https://aka.ms/diagnostic_settings_log_retention).
# https://github.com/hashicorp/terraform-provider-azurerm/pull/23260
# https://github.com/hashicorp/terraform-provider-azurerm/issues/23051
retention_policy {
enabled = true
}
}
}
}

0 comments on commit c9d8d0e

Please sign in to comment.