Skip to content

Commit

Permalink
Remove dynamic memory alert since it is unhelpful (#233)
Browse files Browse the repository at this point in the history
Co-authored-by: Bella L. Quintero <[email protected]>
Co-authored-by: jcrichlake <[email protected]>
Co-authored-by: James Gilmore <[email protected]>
Co-authored-by: Samuel Aquino <[email protected]>
  • Loading branch information
5 people authored Nov 22, 2024
1 parent 46677bf commit 8fa3a04
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 @@ -178,47 +178,6 @@ resource "azurerm_monitor_metric_alert" "low_instance_count_alert" {
}
}

resource "azurerm_monitor_metric_alert" "rs_sftp_dynamic_memory_alert" {
count = local.non_pr_environment ? 1 : 0
name = "cdc-rs-sftp-${var.environment}-dynamic-memory-alert"
resource_group_name = data.azurerm_resource_group.group.name
scopes = [azurerm_linux_web_app.sftp.id]
description = "Alert when memory usage is high on CDC RS SFTP"
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 = data.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" "rs_sftp_memory_alert" {
count = local.non_pr_environment ? 1 : 0
name = "cdc-rs-sftp-${var.environment}-memory-alert"
Expand Down

0 comments on commit 8fa3a04

Please sign in to comment.