Skip to content

Commit

Permalink
add metric filter for RUM uncaught errors (#1287)
Browse files Browse the repository at this point in the history
  • Loading branch information
codemonkey800 committed Oct 19, 2023
1 parent 4b7e180 commit f5c4c45
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 8 deletions.
39 changes: 31 additions & 8 deletions .happy/terraform/modules/cloudwatch-alarm/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -99,11 +99,38 @@ resource aws_cloudwatch_log_metric_filter frontend_error {
}
}

resource aws_cloudwatch_log_metric_filter frontend_uncaught_error {
name = "${var.stack_name}-frontend-uncaught-error"
log_group_name = var.frontend_rum_log_group_name
pattern = join(" ", [
"\"com.amazon.rum.js_error_event\"",
# Errors that occur when fetching RUM credentials. Safe to ignore because
# this is due to the user's network environment.
"-\"CWR: Failed to retrieve Cognito identity\"",
"-\"CWR: Failed to retrieve credentials\"",
# Below errors are safe to ignore because they occur intermittently and do
# not impact the user's abilty to use the napari hub
"-\"ResizeObserver loop\"",
"-\"Script error\"",
"-\"The provided `href`\"",
"-\"The request is not allowed by the user agent\"",
])
count = var.metrics_enabled ? 1 : 0

metric_transformation {
name = "${var.stack_name}-frontend-uncaught-error"
namespace = local.metrics_namespace
value = "1"
unit = "Count"
}
}

locals {
backend_api_500_log_metric_name = var.metrics_enabled ? aws_cloudwatch_log_metric_filter.backend_api_500_log_metric[0].name : "backend_api_500_log_metric"
data_workflows_metrics_update_successful_name = var.metrics_enabled ? aws_cloudwatch_log_metric_filter.data_workflows_metrics_update_successful[0].name : "data_workflows_metrics_update_successful"
data_workflows_plugin_update_successful_name = var.metrics_enabled ? aws_cloudwatch_log_metric_filter.data_workflows_plugin_update_successful[0].name : "data_workflows_plugin_update_successful"
frontend_error_name = var.metrics_enabled ? aws_cloudwatch_log_metric_filter.frontend_error[0].name : "frontend_error"
frontend_uncaught_error_name = var.metrics_enabled ? aws_cloudwatch_log_metric_filter.frontend_uncaught_error[0].name : "frontend_uncaught_error"
}

module backend_api_500_alarm {
Expand Down Expand Up @@ -282,18 +309,14 @@ module frontend_uncaught_error_alarm {
alarm_name = "${var.stack_name}-frontend-uncaught-error-alarm"
comparison_operator = "GreaterThanOrEqualToThreshold"
create_metric_alarm = var.alarms_enabled
datapoints_to_alarm = 1
evaluation_periods = 1
metric_name = "JsErrorCount"
namespace = "AWS/RUM"
datapoints_to_alarm = 2
evaluation_periods = 3
metric_name = local.frontend_uncaught_error_name
namespace = local.metrics_namespace
period = 60
statistic = "Sum"
tags = var.tags
threshold = 1

dimensions = {
application_name = var.frontend_rum_app_name
}
}

module frontend_error_alarm {
Expand Down
5 changes: 5 additions & 0 deletions .happy/terraform/modules/cloudwatch-alarm/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -58,3 +58,8 @@ variable frontend_rum_app_name {
type = string
description = "App name for frontend RUM monitor"
}

variable frontend_rum_log_group_name {
type = string
description = "Log group name for frontend RUM monitor"
}
1 change: 1 addition & 0 deletions .happy/terraform/modules/ecs-stack/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -686,4 +686,5 @@ module alarm_and_monitoring {
tags = var.tags
frontend_log_group_name = module.frontend_service.cloudwatch_log_group_name
frontend_rum_app_name = local.cloudwatch_rum_config.app_name
frontend_rum_log_group_name = local.cloudwatch_rum_config.log_group_name
}

0 comments on commit f5c4c45

Please sign in to comment.