removed the attribute retention_policy due to it being deprecated and…

… data.log_analytics_workspace_id covers this configuration

ops/dev4/_data.tf

@@ -196,6 +196,7 @@ data "azurerm_key_vault_secret" "report_stream_exception_callback_token" {
 data "azurerm_log_analytics_workspace" "log_analytics" {
   name                = "simple-report-log-workspace-global"
   resource_group_name = data.azurerm_resource_group.rg_global.name
+  retention_in_days = 7
 }
 
 data "azurerm_application_insights" "app_insights" {

ops/services/alerts/app_service_metrics/exceptions.tf

@@ -6,18 +6,10 @@ resource "azurerm_monitor_diagnostic_setting" "collect_appserviceconsolelogs" {
 
   enabled_log {
     category = "AppServiceConsoleLogs"
-    retention_policy {
-      days    = 7
-      enabled = true
-    }
   }
   metric {
     category = "AllMetrics"
     enabled  = false
-    retention_policy {
-      days    = 0
-      enabled = false
-    }
   }
 }
 

ops/services/app_gateway/main.tf

@@ -487,10 +487,6 @@ resource "azurerm_monitor_diagnostic_setting" "logs_metrics" {
     ]
     content {
       category = enabled_log.value
-
-      retention_policy {
-        enabled = false
-      }
     }
   }
 
@@ -500,10 +496,6 @@ resource "azurerm_monitor_diagnostic_setting" "logs_metrics" {
     ]
     content {
       category = metric.value
-
-      retention_policy {
-        enabled = false
-      }
     }
   }
 }

ops/services/postgres_db/monitor.tf

@@ -5,19 +5,11 @@ resource "azurerm_monitor_diagnostic_setting" "postgres" {
 
   enabled_log {
     category = "PostgreSQLLogs"
-
-    retention_policy {
-      enabled = false
-    }
   }
 
   metric {
     category = "AllMetrics"
     enabled  = true
-
-    retention_policy {
-      enabled = false
-    }
   }
 }