-
Notifications
You must be signed in to change notification settings - Fork 780
Closed
Labels
Description
Feature Request
Summary
Currently when the BlockSync reactor is being executed, it only produces one metric called Syncing which flags whether BlockSync is catching up or not.
Problem Definition
Having metrics for the latest block that BlockSync has processed would be useful for
Proposal
Add new metrics for:
- the height of the latest block processed by BlockSync,
- number and size of transactions of the latest block, and
- total number of transactions processed.
The new metrics should be in Prometheus namespace for BlockSync. We should not reuse Consensus' metrics.
Also, when BlockSync finishes and switches to Consensus, the metrics should not be registered anymore, as they would be not relevant.
Note that there used to be an issue for this feature that was closed for v0.36.x, but it's missing from main. Only the unused method RecordConsMetrics was left, which should be removed.
Reactions are currently unavailable