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
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Herald added subscribers: tomek, ashoat. · View Herald TranscriptTue, Sep 24, 3:19 PM2024-09-24 15:19:27 (UTC-7)
Harbormaster completed remote builds in B31833: Diff 44519.Tue, Sep 24, 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.Tue, Sep 24, 7:27 PM2024-09-24 19:27:09 (UTC-7)
This revision is now accepted and ready to land.Wed, Sep 25, 12:38 PM2024-09-25 12:38:09 (UTC-7)
will added a child revision: D13472: [terraform] upgrade staging tunnelbroker to 0.17-staging.Wed, Sep 25, 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.Wed, Sep 25, 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 WhyWed, Sep 25, 3:02 PM2024-09-25 15:02:41 (UTC-7)
This revision was automatically updated to reflect the committed changes.