Page MenuHomePhabricator
Differential D12579

[terraform] service connection error alarms
ClosedPublic
Actions

Authored by will on Wed, Jun 26, 7:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Unknown Object (File)
Thu, Jun 27, 11:06 AM
Subscribers
ashoat
tomek

Details

Reviewers
bartek
varun
Commits
rCOMMc467ef6439c5: [terraform] service connection error alarms
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 created this revision.Wed, Jun 26, 7:36 AM
Harbormaster completed remote builds in B29930: Diff 41707.Wed, Jun 26, 7:53 AM
will requested review of this revision.Wed, Jun 26, 7:53 AM
will planned changes to this revision.Wed, Jun 26, 8:45 AM

We actually do want this as "dns error"

will updated this revision to Diff 41710.Wed, Jun 26, 9:17 AM

change pattern to "dns error"

will edited the test plan for this revision. (Show Details)Wed, Jun 26, 9:18 AM
will edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B29933: Diff 41710.Wed, Jun 26, 9:33 AM
will updated this revision to Diff 41739.Wed, Jun 26, 2:58 PM

fix filters and log group names

will edited the test plan for this revision. (Show Details)Wed, Jun 26, 3:00 PM
will edited the test plan for this revision. (Show Details)
will updated this revision to Diff 41742.Wed, Jun 26, 3:15 PM

add email subscription

Harbormaster completed remote builds in B29944: Diff 41739.Wed, Jun 26, 3:16 PM
Harbormaster completed remote builds in B29945: Diff 41742.Wed, Jun 26, 3:32 PM
bartek accepted this revision.Thu, Jun 27, 6:44 AM

Nice!

This revision is now accepted and ready to land.Thu, Jun 27, 6:44 AM
Closed by commit rCOMMc467ef6439c5: [terraform] service connection error alarms (authored by will). · Explain WhyThu, Jun 27, 10:06 AM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMMc467ef6439c5: [terraform] service connection error alarms.

Revision Contents
Changeset List

PathSize
services/
terraform/
remote/
51 lines

Diff 41759

View Options

services/terraform/remote/aws_cloudwatch_alarms.tf

Loading...