Skip to content

Commit

Permalink
Remove / char from resource name (#426)
Browse files Browse the repository at this point in the history
* This breaks Azure API requests because / is a URL escape character
  • Loading branch information
DrizzlyOwl authored Sep 30, 2024
1 parent ae00da9 commit b486d6d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions monitor.tf
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ resource "azurerm_monitor_metric_alert" "count" {
resource "azurerm_monitor_metric_alert" "sql_cpu" {
count = local.enable_monitoring && local.enable_mssql_database ? 1 : 0

name = "SQL Database CPU - ${azurerm_mssql_server.default[0].name}/${azurerm_mssql_database.default[0].name}"
name = "SQL Database CPU - ${azurerm_mssql_database.default[0].name}"
resource_group_name = local.resource_group.name
scopes = [azurerm_mssql_database.default[0].id]
description = "SQL Database ${azurerm_mssql_server.default[0].name}/${azurerm_mssql_database.default[0].name} is consuming more than 80% of CPU"
Expand All @@ -149,7 +149,7 @@ resource "azurerm_monitor_metric_alert" "sql_cpu" {
resource "azurerm_monitor_metric_alert" "sql_dtu" {
count = local.enable_monitoring && local.enable_mssql_database ? 1 : 0

name = "SQL Database DTU - ${azurerm_mssql_server.default[0].name}/${azurerm_mssql_database.default[0].name}"
name = "SQL Database DTU - ${azurerm_mssql_database.default[0].name}"
resource_group_name = local.resource_group.name
scopes = [azurerm_mssql_database.default[0].id]
description = "SQL Database ${azurerm_mssql_server.default[0].name}/${azurerm_mssql_database.default[0].name} is consuming more than 80% of available DTUs"
Expand Down

0 comments on commit b486d6d

Please sign in to comment.