-
Notifications
You must be signed in to change notification settings - Fork 87
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
ref(metrics): Send network outage metric on healthcheck endpoint hit #856
ref(metrics): Send network outage metric on healthcheck endpoint hit #856
Conversation
0be2349
to
5582459
Compare
5582459
to
0e415c3
Compare
0e415c3
to
a057366
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would do it the other way around: push network outage state from the upstream actor into the healthcheck actor.
The original idea with the healthcheck actor was that one would not overload any other actor's mailboxes with healthcheck-related messages.
I wonder whether emitting the gauge so often is even necessary. Perhaps we can set it only if the state changes, and then only frmo within the upstream actor?
dc8cb40
to
c790cd6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
No description provided.