Skip to content

feat(l1): use grafana and prometheus outside of docker #3170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 55 commits into from
Closed

feat(l1): use grafana and prometheus outside of docker #3170

wants to merge 55 commits into from

Conversation

rodrigo-o
Copy link
Collaborator

@rodrigo-o rodrigo-o commented Jun 13, 2025 •
edited by mpaulucci
Loading

Motivation

Description

Closes #3849

rodrigo-o and others added 30 commits June 5, 2025 15:34
@rodrigo-o
Initial Test - Squashed commit of the following:
02d3e4a 
commit 3b2faea
Author: Rodrigo Oliveri <[email protected]>
Date:   Wed Jun 4 20:20:06 2025 -0300

    Initial tests

commit 9644f1c
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 13:21:29 2025 -0300

    Added info syncing logs and fixes to makefile on MacOS

commit 764a23d
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 12:50:22 2025 -0300

    Readded readme file

commit 4183add
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 12:44:09 2025 -0300

    Remove accidentally pushed file

commit bb36195
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 12:42:34 2025 -0300

    Added readme and minor changes to the makefile

commit b050979
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 12:41:09 2025 -0300

    Added readme and minor changes to the makefile

commit 3f4afd0
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 12:07:18 2025 -0300

    Fixed unescaped space in path

commit ec499c6
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 11:49:17 2025 -0300

    Fixed conditional syntax

commit 773bfd5
Author: SDartayet <[email protected]>
Date:   Wed Jun 4 11:45:27 2025 -0300

    Made makefile OS agnostic

commit 168d867
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 18:58:22 2025 -0300

    Added help target and minor corrections

commit 8b2ad93
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 18:46:25 2025 -0300

    Added network env variable check to gen_jwt target

commit 0261f2b
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 18:44:54 2025 -0300

    Minor correction

commit 139fb88
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 18:38:44 2025 -0300

    Made some fixes

commit 9ee2994
Merge: 7caadda f27cfff
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 18:14:00 2025 -0300

    Merge branch 'main' into sync-tools

commit 7caadda
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 17:49:21 2025 -0300

    Fixed bug with sync block number env variable

commit 89c48ea
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 17:23:41 2025 -0300

    Fixed datadir in lighthouse targets

commit 5b5a4b9
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 17:14:40 2025 -0300

    Fixed typos

commit f0d1f46
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 17:10:03 2025 -0300

    Reworked jwt and datadir paths

commit 7250c11
Author: SDartayet <[email protected]>
Date:   Tue Jun 3 16:47:06 2025 -0300

    Minor path corrections

commit 04f6bce
Author: SDartayet <[email protected]>
Date:   Mon Jun 2 18:58:03 2025 -0300

    Renamed env variables in code and added Makefile functionality for them

commit cdff554
Author: SDartayet <[email protected]>
Date:   Mon Jun 2 16:01:54 2025 -0300

    Moved to tooling folder and updated Makefile

commit 0c8c8c3
Author: SDartayet <[email protected]>
Date:   Wed May 28 15:09:03 2025 -0300

    Revert "Added makefile with sync testing tools"

    This reverts commit 1303a6f.

commit b862b32
Author: SDartayet <[email protected]>
Date:   Wed May 28 15:08:47 2025 -0300

    Added makefile with sync testing tools
@rodrigo-o
Updated dashboard to revision 5
82a0338
@rodrigo-o
Merge remote-tracking branch 'origin/main' into grafana-l1-metric-exp…
08bb214 
…orter
@rodrigo-o
clean yamls and add make targets
a5ba6b5
@rodrigo-o
fix gateway
01e5763
@rodrigo-o
Merge branch 'main' into grafana-l1-metric-exporter
7bc987b
@rodrigo-o
enable tx_pool module in metrics exporter
85ea8e5
@rodrigo-o
Added txpoll_status to the RPC API
233fc8e
@rodrigo-o
Merge remote-tracking branch 'origin/main' into grafana-l1-metric-exp…
228a804 
…orter
@SDartayet
Merge branch 'main' into grafana-l1-metric-exporter
b581b19
@rodrigo-o
Linting
ea39997
@rodrigo-o
Some cleanup + renaming
cf95edd
@rodrigo-o
Merge branch 'main' into grafana-l1-metric-exporter
3cc2556
@rodrigo-o
Fix metrics override
41ec67f
@rodrigo-o
Actually fix the prometheus config error
a7ec556
@rodrigo-o
Merge remote-tracking branch 'origin/main' into grafana-l1-metric-exp…
3763e6e 
…orter
@rodrigo-o
Merge branch 'main' into grafana-l1-metric-exporter
303f922
@SDartayet
feat(l1): L1 throughput export (#3106)
cc60437 
**Motivation**

Adding throughput metrics to the L1 metrics exporter.

**Description**

A new pane was added with the L1 block throughput, with the necessary
code in the `crates/blockchain/blockchain.rs` file.
@rodrigo-o
Merge branch 'main' into grafana-l1-metric-exporter
c5cd701
@rodrigo-o
add metrics feature to make targets
232bab1
@SDartayet
Feature flag fix
e76e723
@SDartayet
Minor fixes
0373b4b
@SDartayet
Removed unnecessary feature flag use
5c40b09
@SDartayet
minor fix
e22d6d8
@rodrigo-o
Merge remote-tracking branch 'origin/main' into HEAD
888dedc
@rodrigo-o
Add host mapping to prometheus
18714bb
@SDartayet
minor fix
77a72a7
@SDartayet
Fixed f64 type
ed48f80
@rodrigo-o
Be consistent with previous code regarding _f64 suffix
c7625f7
@SDartayet
Removed flawed blocks per second metric
331fb70
@rodrigo-o rodrigo-o changed the title Server hoodi testing feat(l1): use grafana and prometheus outside of docker Jun 13, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 14, 2025 •
edited
Loading

Lines of code report

Total lines added: 60
Total lines removed: 0
Total lines changed: 60

Detailed view 
+----------------------------------------+-------+------+
| File                                   | Lines | Diff |
+----------------------------------------+-------+------+
| ethrex/crates/blockchain/blockchain.rs | 744   | +47  |
+----------------------------------------+-------+------+
| ethrex/crates/networking/p2p/sync.rs   | 588   | +13  |
+----------------------------------------+-------+------+

Base automatically changed from grafana-l1-metric-exporter to main June 17, 2025 14:48
@JereSalo JereSalo added the L1 Ethereum client label Jul 3, 2025
@JereSalo JereSalo moved this to In Progress in ethrex_l1 Jul 3, 2025
@mpaulucci mpaulucci mentioned this pull request Jul 28, 2025
Open
@mpaulucci
Copy link
Collaborator

Closing for now

@mpaulucci mpaulucci closed this Jul 28, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in ethrex_l1 Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

@rodrigo-o rodrigo-o

Labels
L1 Ethereum client
Projects
ethrex_l1
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add ability to configure grafana/prometheus outside of docker
4 participants
@rodrigo-o @mpaulucci @JereSalo @SDartayet