Skip to content

chore: Add network-monitor image and update compose with tempo and grafana#1948

Merged
sergerad merged 12 commits intonextfrom
sergerad-docker-exts
Apr 20, 2026
Merged

chore: Add network-monitor image and update compose with tempo and grafana#1948
sergerad merged 12 commits intonextfrom
sergerad-docker-exts

Conversation

@sergerad
Copy link
Copy Markdown
Collaborator

Relates to #1886.

@sergerad sergerad changed the title chore: Add network-monitor image and update compose with tmepo and grafana chore: Add network-monitor image and update compose with tempo and grafana Apr 16, 2026
Comment thread crates/utils/src/logging.rs Outdated
Comment thread compose/telemetry.yml
@sergerad sergerad added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Apr 17, 2026
kkovaacs and others added 4 commits April 17, 2026 08:27
These are required to get the Drilldown/Traces page work in Grafana.
Tempo has implemented some changes that make the metrics_generator
configuration broken with `latest`.
Copy link
Copy Markdown
Collaborator

@SantiagoPittella SantiagoPittella left a comment

Choose a reason for hiding this comment

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

Overall looks good and works properly. The default dashboard can be improved but we can tackle that later I guess

Comment thread docker-compose.yml Outdated
Comment on lines +153 to +160
environment:
- MIDEN_MONITOR_RPC_URL=http://rpc:57291
- MIDEN_MONITOR_PORT=3001
- MIDEN_MONITOR_NETWORK_NAME=Localhost
- MIDEN_MONITOR_DISABLE_NTX_SERVICE=true
- MIDEN_MONITOR_ENABLE_OTEL=true
- OTEL_EXPORTER_OTLP_ENDPOINT=http://tempo:4317
- OTEL_SERVICE_NAME=monitor
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We probably would want to enable some more checks, like validator and network transactions. This can be done in a follow up PR, if you create an issue feel free to assign it to me.

Copy link
Copy Markdown
Contributor

@kkovaacs kkovaacs left a comment

Choose a reason for hiding this comment

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

Works great for me. Thanks!

@sergerad sergerad merged commit a6288e7 into next Apr 20, 2026
19 checks passed
@sergerad sergerad deleted the sergerad-docker-exts branch April 20, 2026 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants