Skip to content

Conversation

@omerfirmak
Copy link
Member

Switches to using counters so that the gauges don't cause any information to be lost. Counters can be used to calculate all sorts of metrics on Grafana. Which is also why min/avg/max logic is removed to make things simple and small here.

Switches to using counters so that the gauges don't cause any information
to be lost. Counters can be used to calculate all sorts of metrics on Grafana.
Which is also why min/avg/max logic is removed to make things simple and small here.
@omerfirmak omerfirmak force-pushed the stateless-depth-metric branch from ff19c05 to 1d0c7df Compare September 2, 2025 08:49
@gballet gballet added this to the 1.16.4 milestone Sep 2, 2025
Copy link
Member

@gballet gballet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The initial plan was to only have it for the storage nodes, but I think it makes sense to have it for the tree as well, and let the person doing analytics process the information themselves.

@gballet gballet merged commit c36f7be into ethereum:master Sep 2, 2025
5 of 6 checks passed
Sahil-4555 pushed a commit to Sahil-4555/go-ethereum that referenced this pull request Oct 12, 2025
…um#32533)

Switches to using counters so that the gauges don't cause any
information to be lost. Counters can be used to calculate all sorts of
metrics on Grafana. Which is also why min/avg/max logic is removed to
make things simple and small here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants