Skip to content

Conversation

@mablr
Copy link
Contributor

@mablr mablr commented Oct 22, 2025

Closes #19229

This spawns a task to push metrics to prometheus following configurable interval (default 5sec).
Added CLI flags for push gateway url and push interval.

Copy link
Contributor Author

@mablr mablr left a comment

Choose a reason for hiding this comment

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

I tested it locally like this :

docker run -d -p 9091:9091 prom/pushgateway

reth node --metrics=0.0.0.0:9001 --metrics.push.url=http://localhost:9091/metrics/job/test

curl http://localhost:9091/metrics

It seems to push correctly.

I'm wondering if it's relevant to add a test that spawns a HTTP server in order to inspect the PUT requests sent by the MetricServer. Wdyt ?

@mablr mablr marked this pull request as ready for review October 22, 2025 22:01
Copy link
Collaborator

@shekhirin shekhirin left a comment

Choose a reason for hiding this comment

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

only two comments, this is basically what we need, yeah!

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Oct 23, 2025
@shekhirin shekhirin added C-enhancement New feature or request A-observability Related to tracing, metrics, logs and other observability tools labels Oct 23, 2025
@mablr mablr requested a review from shekhirin October 23, 2025 09:04
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

this lgtm

@Rjected likely has more context here

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

Labels

A-observability Related to tracing, metrics, logs and other observability tools C-enhancement New feature or request

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Support pushing Prometheus metrics to a remote URL

3 participants