From 93219903d0b4a33ee536a64477c87e93f64ba50e Mon Sep 17 00:00:00 2001
From: Jacob Winch <jacob.winch@guardian.co.uk>
Date: Thu, 10 Jan 2019 14:39:00 +0000
Subject: [PATCH] Remove 4XX alarm

4XX includes payment errors (402s), which should not trigger alerts.
---
 handlers/new-product-api/cfn.yaml | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/handlers/new-product-api/cfn.yaml b/handlers/new-product-api/cfn.yaml
index c9b6ee3d6a..f9cda33f7b 100644
--- a/handlers/new-product-api/cfn.yaml
+++ b/handlers/new-product-api/cfn.yaml
@@ -326,26 +326,3 @@ Resources:
         Statistic: Sum
         Threshold: 5
         TreatMissingData: notBreaching
-    4xxApiAlarm:
-        Type: AWS::CloudWatch::Alarm
-        Condition: CreateProdMonitoring
-        Properties:
-          AlarmActions:
-          - !Sub arn:aws:sns:${AWS::Region}:${AWS::AccountId}:subscriptions_dev
-          AlarmName:
-            !Sub
-             - 4XX rate from ${ApiName}
-             - { ApiName: !FindInMap [StageMap, !Ref Stage, ApiName] }
-          ComparisonOperator: GreaterThanThreshold
-          Dimensions:
-            - Name: ApiName
-              Value: !FindInMap [StageMap, !Ref Stage, ApiName]
-            - Name: Stage
-              Value: !Sub ${Stage}
-          EvaluationPeriods: 1
-          MetricName: 4XXError
-          Namespace: AWS/ApiGateway
-          Period: 3600
-          Statistic: Sum
-          Threshold: 5
-          TreatMissingData: notBreaching
\ No newline at end of file