Add Health Check to Roles #1786
Closed
lucasbalieiro
started this conversation in
Ideas
Replies: 1 comment
|
yeah strictly speaking maybe this could generate false positives, but we would be able to quickly spot them by checking logs so for now I think it's ok, we can iterate and improve over time |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
I'm building a small tool that captures the latest commits on the main branch of the SRI repo, rebuilds the code, and runs the Roles as defined in the tutorial: https://stratumprotocol.org/blog/getting-started#ii-getting-started---running-all-roles
The idea is to extend the CI flow with a 24/7 process that continuously runs everything. This may helps us catch regressions early, like if we accidentally break something described in the tutorial.
One of the key features I want to add is a way to alert us when something breaks.
Right now, I check if the last log written by any Role is older than 5 minutes. If it is, I mark it with a yellow indicator (and plan to send a notification, e.g. to Discord):

This also led to PR #1785.
However, this approach feels a bit fragile. It doesn’t really confirm that the Role itself is healthy—just that something was recently logged.
Do you have suggestions for a better health check strategy?
Should we think about adding an explicit health signal to the Roles?
Or is monitoring the timestamp of the last log good enough indicator?
All reactions