-
Notifications
You must be signed in to change notification settings - Fork 46
feat(cdk): wire DLQ CloudWatch alarms to SNS topic for alerting #228
Copy link
Copy link
Open
Labels
enhancementNew feature or requestNew feature or requestinfra-cdkCDK stacks/constructs, bootstrap, deploy topology, tags, IAM wiring, teardownCDK stacks/constructs, bootstrap, deploy topology, tags, IAM wiring, teardownobservabilityTracing, attribution, dashboards, metrics, alarms, telemetry redactionTracing, attribution, dashboards, metrics, alarms, telemetry redactionv1Version 1Version 1
Description
Activity
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestinfra-cdkCDK stacks/constructs, bootstrap, deploy topology, tags, IAM wiring, teardownCDK stacks/constructs, bootstrap, deploy topology, tags, IAM wiring, teardownobservabilityTracing, attribution, dashboards, metrics, alarms, telemetry redactionTracing, attribution, dashboards, metrics, alarms, telemetry redactionv1Version 1Version 1
Context
PR #208 added CloudWatch alarms on
FanOutDlqandApprovalMetricsPublisherDlq(threshold:ApproximateNumberOfMessagesVisible >= 1, 5-min window). The alarms are exposed aspublic readonly dlqAlarmon each construct but are not yet wired to any notification channel — they fire silently in the CloudWatch console.Proposal
Create an SNS topic (or accept one as a construct prop) and add alarm actions so operators get notified (email, PagerDuty, Slack webhook, etc.) when poison-pill records land in a DLQ.
Acceptance criteria
FanOutConsumer.dlqAlarmandApprovalMetricsPublisherConsumer.dlqAlarmhave an alarm action targeting the topiccdk synthproduces the expectedAWS::CloudWatch::Alarm→AWS::SNS::TopicrelationshipDesign considerations
OperationalAlarmsTopicis simpler and matches the common pattern.notificationEndpointsstack parameter.References