Skip to content

chore: Use custom infra similar prometheus image#3062

Open
ercecan wants to merge 11 commits intonightlyfrom
use-custom-infra-similar-prometheus-image
Open

chore: Use custom infra similar prometheus image#3062
ercecan wants to merge 11 commits intonightlyfrom
use-custom-infra-similar-prometheus-image

Conversation

@ercecan
Copy link
Member

@ercecan ercecan commented Dec 4, 2025

Description

Using a script similar to one in our production infra for generating prometheus.yml file we can now directly import locally created dashboards into production grafana

Copilot AI review requested due to automatic review settings December 4, 2025 11:32
@ercecan ercecan requested a review from a team as a code owner December 4, 2025 11:32
@auto-assign auto-assign bot requested a review from kpp December 4, 2025 11:32
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

⚠️ Changelog entry missing

No changes detected in CHANGELOG.md.

Recommendation:

chore: Use custom infra similar prometheus image

Please add an entry to the CHANGELOG.md or dismiss this if the change doesn't require documentation.

To dismiss: Reply with /skip-changelog in any comment.

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

This PR modernizes the local telemetry infrastructure to align with production environments by introducing a Python-based Prometheus configuration generator. This enables seamless import/export of Grafana dashboards between local development and production environments.

Key Changes

  • Introduces a custom Prometheus Docker image with Python-based configuration generation
  • Updates Grafana datasource configuration to match production UID for dashboard portability
  • Standardizes job names and adds network/environment labels for multi-environment support

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 15 comments.

Show a summary per file
File Description
docker/prometheus/Dockerfile New custom Prometheus image with Python and PyYAML for dynamic config generation
docker/prometheus/init.py Python script that generates prometheus.yml with consistent labeling (net_name, env_name)
docker/prometheus/start.sh Entrypoint script that runs init.py before starting Prometheus
docker/telemetry/prometheus.yml Generated Prometheus config with standardized job names and relabeling rules
docker/telemetry/grafana/datasources.yml New datasource configuration with hardcoded production UID for dashboard compatibility
docker/docker-compose.telemetry.yaml Updated to use custom Prometheus image and provision Grafana datasources
docker/README.md Documentation for local setup matching production infrastructure
resources/grafana/sequencer.dashboard.json Updated dashboard with consistent datasource UIDs and net_name variable filtering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ercecan ercecan requested a review from eyusufatik December 5, 2025 15:30
@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.6%. Comparing base (cfb5069) to head (e32fa43).
⚠️ Report is 1 commits behind head on nightly.

Additional details and impacted files

see 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

kpp
kpp previously approved these changes Dec 9, 2025
Copy link
Member

Choose a reason for hiding this comment

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

on "user" dir no need for batch prover, light client prover and sequencer dashboards.

if the reason you put them is we are gonna use those dashboards on our local envs (regtest) let's set up a different directory for them

Copy link
Member Author

Choose a reason for hiding this comment

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

us developers are not going to use them, only external node runners will use them, so I will just keep full node then

Copy link
Member Author

Choose a reason for hiding this comment

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

however I believe people like auditors or someone who wanna dive deep and run the stack locally can still use them?

@ercecan ercecan requested a review from eyusufatik December 9, 2025 14:33
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.

4 participants