-
Notifications
You must be signed in to change notification settings - Fork 332
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
[nodeinfo] add p2p address into metrics #4143
base: master
Are you sure you want to change the base?
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4143 +/- ##
==========================================
+ Coverage 75.38% 76.30% +0.92%
==========================================
Files 303 339 +36
Lines 25923 28870 +2947
==========================================
+ Hits 19541 22030 +2489
- Misses 5360 5734 +374
- Partials 1022 1106 +84 ☔ View full report in Codecov by Sentry. |
Help: "height info of node", | ||
}, | ||
[]string{"address", "version"}, | ||
var ( |
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.
one gauge is enough?
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.
it is for grafana transition. I also thought just add p2pID
into existing gauge, but would that cause data error on grafana? nodes with new version have 3 labels in the gauge, while nodes with current version still have 2 labels in the gauge
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 don't think there would be error on grafana, but you can test it.
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.
Should work well using one gauge
Description
add p2p address into metrics. In case of need, we can follow the p2p address to find the node's IP address
Fixes #3956
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: