-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🔊 setup sns topic for alarm notifications; notify of stopped containers #406
Conversation
Codecov Report
@@ Coverage Diff @@
## main #406 +/- ##
=======================================
Coverage 97.09% 97.09%
=======================================
Files 81 81
Lines 1581 1581
Branches 213 213
=======================================
Hits 1535 1535
Misses 37 37
Partials 9 9 Continue to review full report at Codecov.
|
Seems lambda's free tier is more than enough to cover alarms. Moving to draft, I'll fiddle with publishing to a discord webhook instead. |
Now it's all lambda. Neat. |
Revisiting due to the merge, I'm not sure it's a great alarm. Stopped containers still get restarted by ECS, so it's still an alarm we can't really action on. Will re-think in the future. Just closing. |
Summary
Note: maybe do #423 first
I guess fixes #384, though we can add more alarms. If you have better ideas for what to alarm on, let me know.
Spent a while looking into all the different alarming options:
I figured:
So, being notified of unintentionally stopped ecs containers seemed like enough, at least for starters. See tutorial I mostly followed.
Note this doesn't actually add subscriptions, we can do that in follow up commits.
It'd be specifying theWe'll need a new webhook to publish to.AlarmSubscription*
cloudformation parameters, which would likely be github secrets.Tested manually by creating the topic, rule and subscribing myself. I then ssh'd into the ec2 host and did
docker kill
.Received a single email for the dead instance.See#duckbutt
channel in the beta server.Checklist
format
in the repository rootpytest
in the repository root