Skip to content

Conversation

@rodrigo-o
Copy link
Collaborator

Motivation

We were seeing old networks in our grafana dashboards

Description

This RP just restarts ethereum-metrics-exporter container to have the new network everytime we restart the node

@rodrigo-o rodrigo-o requested a review from a team as a code owner October 14, 2025 00:09
Copilot AI review requested due to automatic review settings October 14, 2025 00:09
@github-actions github-actions bot added the L1 Ethereum client label Oct 14, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Restarts the ethereum-metrics-exporter container when starting the node using sync tooling to ensure old network data doesn't persist in Grafana dashboards.

  • Adds docker stop command to explicitly stop the metrics-ethereum-metrics-exporter-1 container before starting services
  • Ensures fresh network data is displayed in monitoring dashboards when switching networks

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-project-automation github-project-automation bot moved this to In Review in ethrex_l1 Oct 14, 2025
@rodrigo-o rodrigo-o added this pull request to the merge queue Oct 17, 2025
Merged via the queue into main with commit 47c0648 Oct 17, 2025
29 checks passed
@rodrigo-o rodrigo-o deleted the reset-ethereum-exporter-on-node-start branch October 17, 2025 02:21
@github-project-automation github-project-automation bot moved this from In Review to Done in ethrex_l1 Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L1 Ethereum client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants