Skip to content

Commit

Permalink
fix: update CSP error alarm to catch Exceptions only (#429)
Browse files Browse the repository at this point in the history
Update the CSP CloudWatch `error` metric alarm to catch
exceptions rather than CSP violations that include the word `Error`.
  • Loading branch information
patheard authored Feb 19, 2024
1 parent e0e6942 commit 56e7fc0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion terragrunt/aws/csp_violation_report_service/alarms.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
resource "aws_cloudwatch_log_metric_filter" "csp_report_error" {
name = "CSPReportError"
pattern = "?ERROR ?Error"
pattern = "?ERROR ?Traceback"
log_group_name = local.csp_reports_log_group_name

metric_transformation {
Expand Down
2 changes: 1 addition & 1 deletion terragrunt/aws/csp_violation_report_service/lambda.tf
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
module "csp_reports" {
source = "github.com/cds-snc/terraform-modules//lambda?ref=v8.0.0"
source = "github.com/cds-snc/terraform-modules//lambda?ref=v9.2.1"
name = var.tool_name
ecr_arn = aws_ecr_repository.csp_reports.arn
image_uri = "${aws_ecr_repository.csp_reports.repository_url}:latest"
Expand Down

0 comments on commit 56e7fc0

Please sign in to comment.