Skip to content

Commit

Permalink
Remove dynamic memory alert since it is unhelpful
Browse files Browse the repository at this point in the history
Co-Authored-By: Samuel Aquino <[email protected]>
Co-Authored-By: jcrichlake <[email protected]>
Co-Authored-By: Bella L. Quintero <[email protected]>
Co-Authored-By: James Gilmore <[email protected]>
  • Loading branch information
5 people committed Nov 22, 2024
1 parent a289884 commit 4e589ff
Showing 1 changed file with 0 additions and 41 deletions.
41 changes: 0 additions & 41 deletions operations/template/alert.tf
Original file line number Diff line number Diff line change
Expand Up @@ -217,47 +217,6 @@ resource "azurerm_monitor_metric_alert" "azure_5XX_alert" {
}
}

resource "azurerm_monitor_metric_alert" "dynamic_memory_alert" {
count = local.non_pr_environment ? 1 : 0
name = "cdcti-${var.environment}-dynamic-memory-alert"
resource_group_name = data.azurerm_resource_group.group.name
scopes = [azurerm_linux_web_app.api.id]
description = "This alert checks if the backpack is starting to get heavy but does it in a way that keeps watching how much stuff is added. If it gets too full, it lets you know so you can take action before it becomes a problem."
severity = 2
frequency = "PT5M"
window_size = "PT15M"

dynamic_criteria {
metric_name = "MemoryWorkingSet"
metric_namespace = "Microsoft.Web/sites"
aggregation = "Average"
operator = "GreaterThan"
alert_sensitivity = "Medium"
}

action {
action_group_id = azurerm_monitor_action_group.notify_slack_email[count.index].id
}

lifecycle {
# Ignore changes to tags because the CDC sets these automagically
ignore_changes = [
tags["business_steward"],
tags["center"],
tags["environment"],
tags["escid"],
tags["funding_source"],
tags["pii_data"],
tags["security_compliance"],
tags["security_steward"],
tags["support_group"],
tags["system"],
tags["technical_steward"],
tags["zone"]
]
}
}

resource "azurerm_monitor_metric_alert" "database_memory_alert" {
count = local.non_pr_environment ? 1 : 0
name = "cdcti-${var.environment}-database-memory-alert"
Expand Down

0 comments on commit 4e589ff

Please sign in to comment.