Page MenuHomePhabricator

[terraform] service connection error alarms
ClosedPublic

Authored by will on Jun 26 2024, 7:36 AM.
Tags
None
Referenced Files
F3376886: D12579.id41707.diff
Wed, Nov 27, 2:45 AM
F3376866: D12579.id41739.diff
Wed, Nov 27, 2:39 AM
F3371511: D12579.id41710.diff
Tue, Nov 26, 4:52 AM
Unknown Object (File)
Tue, Nov 12, 1:13 PM
Unknown Object (File)
Tue, Nov 12, 12:55 AM
Unknown Object (File)
Mon, Nov 11, 11:43 PM
Unknown Object (File)
Mon, Nov 11, 11:30 PM
Unknown Object (File)
Mon, Nov 11, 8:24 PM
Subscribers

Details

Summary

This sets up our alarms for errors connecting service A to service B.

We've specified all the relevant log groups in locals. This diff creates alarms that match on dns error in logs. The "dns error" error is included as part of tonic when failing to connect

Test Plan

terraform apply. Tested by launching services configured with improper identity domain name on staging and confirming that alarms are activated

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will requested review of this revision.Jun 26 2024, 7:53 AM
will planned changes to this revision.Jun 26 2024, 8:45 AM

We actually do want this as "dns error"

change pattern to "dns error"

will edited the summary of this revision. (Show Details)

fix filters and log group names

will edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Jun 27 2024, 6:44 AM
This revision was automatically updated to reflect the committed changes.