I'm not 100% if we should be alarming on these given that looking through prior logs, we're spammed with failed FCM notifications. Putting this diff up for feedback
Depends on D13454
Details
Details
Summary
Test Plan
Test that alarms are triggerable later in stack
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Herald added subscribers: tomek, ashoat. · View Herald TranscriptSep 24 2024, 3:19 PM2024-09-24 15:19:27 (UTC-7)
Harbormaster completed remote builds in B31833: Diff 44519.Sep 24 2024, 3:36 PM2024-09-24 15:36:36 (UTC-7)
will retitled this revision from [tunnelbroker] add errorType patterns for alarming on tunnelbroker to [tunnelbroker] add errorType patterns for alarming on tunnelbroker for notifs.Sep 24 2024, 7:27 PM2024-09-24 19:27:09 (UTC-7)
This revision is now accepted and ready to land.Sep 25 2024, 12:38 PM2024-09-25 12:38:09 (UTC-7)
will added a child revision: D13472: [terraform] upgrade staging tunnelbroker to 0.17-staging.Sep 25 2024, 2:26 PM2024-09-25 14:26:38 (UTC-7)
will removed a child revision: D13471: [terraform] set up tracing error alarms for tunnelbroker on cloudwatch.
Harbormaster completed remote builds in B31857: Diff 44568.Sep 25 2024, 2:48 PM2024-09-25 14:48:49 (UTC-7)
Closed by commit rCOMMeeaa71765dec: [tunnelbroker] add errorType patterns for alarming on tunnelbroker for notifs (authored by will). · Explain WhySep 25 2024, 3:02 PM2024-09-25 15:02:41 (UTC-7)
This revision was automatically updated to reflect the committed changes.