Skip to content

[feat] add metrics+health server component for indexer v2 (1/n) - #1278

Closed
larryl3u wants to merge 2 commits into
movement-network:mainfrom
larryl3u:indexer-v2-1
Closed

[feat] add metrics+health server component for indexer v2 (1/n)#1278
larryl3u wants to merge 2 commits into
movement-network:mainfrom
larryl3u:indexer-v2-1

Conversation

@larryl3u

@larryl3u larryl3u commented Jul 16, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Add simple components for health + metrics server.
    • We use a different approach that drives the processors and there is no metrics component in v2. To support the metrics, we create a component here.

Changelog

  • Add health check and metrics endpoint component.

Testing

Locally tested and looks good!

  • Metrics
image
  • Health check
image

Outstanding issues

@larryl3u larryl3u changed the title add metrics+health config for indexer v2. [feat] add metrics+health config for indexer v2 (1/n) Jul 16, 2025
@larryl3u larryl3u changed the title [feat] add metrics+health config for indexer v2 (1/n) [feat] add metrics+health server component for indexer v2 (1/n) Jul 16, 2025

@musitdev musitdev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

My comment is mostly on the fact that if your metric integration is compatible with the Fullnode one.

Comment thread protocol-units/execution/maptos/util/src/config/health_server.rs
Comment thread protocol-units/execution/maptos/util/src/config/metrics_server.rs
Comment thread protocol-units/execution/maptos/util/src/config/metrics_server.rs
@larryl3u

Copy link
Copy Markdown
Contributor Author

move to #1283 ty!

@larryl3u larryl3u closed this Jul 23, 2025
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