-
-
Notifications
You must be signed in to change notification settings - Fork 282
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(beacon-node): add block error metric to gossip handlers #5495
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
@dapplion I have the changes loaded to |
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.
LGTM!
@matthewkeil can you post a screen capture of how the metrics dashboard looks like? |
I did a bit of digging and there weren't any |
Should I add a panel for the other |
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.
all errors should be in the same panel with {{error}}
as legend
dashboards/lodestar_networking.json
Outdated
"uid": "${DS_PROMETHEUS}" | ||
}, | ||
"editorMode": "code", | ||
"expr": "rate(lodestar_gossip_block_process_block_errors{}[$rate_interval]) * 12", |
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.
redundant {}
dashboards/lodestar_networking.json
Outdated
}, | ||
"editorMode": "code", | ||
"expr": "rate(lodestar_gossip_block_process_block_errors{}[$rate_interval]) * 12", | ||
"legendFormat": "{{blockErrorCode}}", |
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.
@matthewkeil legendFormat should be the label in your metric
"legendFormat": "{{blockErrorCode}}", | |
"legendFormat": "{{error}}", |
@matthewkeil the grafana panel in this PR was merged to |
6fb57c0
to
448c0de
Compare
@tuyennhv this is updated. I noticed that the panel still had the old title before it captured all of the errors so i update that to remove the "Unknown Parent" part. I had a bit of trouble with the rebase and needed to cherry pick the commits and force push to get things corrected. Please double check that the changes look ok. |
This reverts commit 56844f5.
🎉 This PR is included in v1.9.0 🎉 |
Motivation
This PR is related to #5481 and captures gossip block error metrics and adds them to a new dashboard. Will help to debug poorly gossiped blocks by peers by exposing the slot, peerId, time seen and error that was raised.
Description
Closes #5481