-
Notifications
You must be signed in to change notification settings - Fork 358
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 batcher queue len and queue size in bytes metrics #1593
base: staging
Are you sure you want to change the base?
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.
You haven't updated the grafana json! Thou I added them myself and it does seem to be working fine.
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.
There is missing to update the grafana dashbaord
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.
Lol yes missing graphana visualizations
Sorry! Grafana dashboard was just added @uri-99 @JuArce @MarcosNicolau |
Worked locally for me! |
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 think these 2 new boxes in grafana are quite informative, yet they ended up quite in the bottom of the dashboard. i would put them a bit higher
Add batcher queue len and queue size in bytes metrics
Description
This PR adds two new metrics for the batcher:
How to test
make batcher_send_sp1_burst BURST_SIZE=23
Type of change
Please delete options that are not relevant.
Checklist
testnet
, everything else tostaging