Skip to content

Commit

Permalink
Fix Firewall Logging (#1870)
Browse files Browse the repository at this point in the history
* switch firewall away from dedicated log tables

* update TF lock

* fix liniting issue with firewall.tf
  • Loading branch information
martinpeck authored May 19, 2022
1 parent 8af9adc commit db7740c
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 30 deletions.
52 changes: 26 additions & 26 deletions templates/core/terraform/.terraform.lock.hcl

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 4 additions & 4 deletions templates/shared_services/firewall/terraform/firewall.tf
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@ resource "azurerm_management_lock" "fw" {
}

resource "azurerm_monitor_diagnostic_setting" "firewall" {
name = "diagnostics-firewall-${var.tre_id}"
target_resource_id = azurerm_firewall.fw.id
log_analytics_workspace_id = data.azurerm_log_analytics_workspace.tre.id
log_analytics_destination_type = "Dedicated"
name = "diagnostics-fw-${var.tre_id}"
target_resource_id = azurerm_firewall.fw.id
log_analytics_workspace_id = data.azurerm_log_analytics_workspace.tre.id
#log_analytics_destination_type = "Dedicated"

log {
category = "AzureFirewallApplicationRule"
Expand Down

0 comments on commit db7740c

Please sign in to comment.