-
Notifications
You must be signed in to change notification settings - Fork 364
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
feat: add operator responses dashboard #1648
Conversation
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.
Tested locally and everything works!
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.
Works nicely! Question: doesn't this metric make the one from #1621 redundant? Since this metric shows the number of responded tasks and the total tasks, the difference in responded tasks can be derived (this could be done via Grafana). Also, this metric updates in real time, whereas the one in #1621 updates only when quorum is reached.
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.
- Sort the values by lower responding operators first
- Show a single color without gradient
Addressed both using a table + override with a gauge display. I had to use the "RETRO LCD" display because with the basic one, the progress bar appears empty when all values are the same: |
Add Operator Responses Dashboard
Description
This PR adds a new Grafana dashboard with the count of the operators responses received in the selected time range:
How to test
You can follow similar steps as #1621 to test the full flow of this PR.
Type of change
Checklist
testnet
, everything else tostaging