Skip to content
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

NodeOS 5.0 Prometheus Metrics should include connid/label for incoming transactions instead of total #1937

Open
jhjhjh94 opened this issue Nov 30, 2023 · 1 comment

Comments

@jhjhjh94
Copy link

nodeos_trxs_incoming_total
nodeos_incoming_elapsed_us_total
nodeos_incoming_us_total
nodeos_incoming_us_block_latency
nodeos_blocks_incoming

Very useful metrics but as of now we can only see the total. In my opinion it’ll be better to see the amount of incoming transactions pointed to a specific connid/label (to see which node sends over the most transaction and also their peering performance)

@jhjhjh94 jhjhjh94 changed the title NodeOS 5.0 Prometheus Metrics should include connid/label instead of total NodeOS 5.0 Prometheus Metrics should include connid/label for incoming transactions instead of total Nov 30, 2023
@bhazzard bhazzard added 👍 lgtm and removed triage labels Nov 30, 2023
@bhazzard
Copy link

Great suggestion. Unfortunately we don't have time left in schedule to get this in for 5.0.0. Tentatively, I'll target 6.0.0 for this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants