Skip to content

Conversation

@garyschulte
Copy link
Collaborator

@garyschulte garyschulte commented Oct 15, 2024

PR Description

Add a metrics gauge tracking shomei head state: shomei_blockchain_head
e.g.:

$ curl -s localhost:9888/metrics|grep shomei
# HELP shomei_blockchain_head current chain head block number which corresponds to shomei state
# TYPE shomei_blockchain_head gauge
shomei_blockchain_head 257373.0


$ curl -s localhost:9888/metrics|grep shomei
# HELP shomei_blockchain_head current chain head block number which corresponds to shomei state
# TYPE shomei_blockchain_head gauge
shomei_blockchain_head 257642.0

also removes the expectation that a MetricsService is a vertx verticle..

Fixed Issue(s)

#99

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@garyschulte garyschulte requested a review from matkt October 15, 2024 00:52
Signed-off-by: garyschulte <[email protected]>
@garyschulte garyschulte force-pushed the feature/shomei-head-metrics branch from a520784 to dad764d Compare October 15, 2024 01:01
Signed-off-by: garyschulte <[email protected]>
Signed-off-by: garyschulte <[email protected]>
@garyschulte garyschulte force-pushed the feature/shomei-head-metrics branch from 59be2ba to 091225d Compare October 15, 2024 14:51
@garyschulte garyschulte merged commit 707f4af into main Oct 15, 2024
10 checks passed
@garyschulte garyschulte deleted the feature/shomei-head-metrics branch October 15, 2024 15:26
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.

3 participants