Skip to content
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.

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

feat(rollup): init telemetry #50

Merged
merged 1 commit into from
Aug 27, 2024
Merged

feat(rollup): init telemetry #50

merged 1 commit into from
Aug 27, 2024

Conversation

merklefruit
Copy link
Collaborator

@merklefruit merklefruit commented Aug 27, 2024

Added the ability to start tracing to stdout with the following options:

  • tracing target: provided by default with RUST_LOG directives. Falls back to hera=info if not provided.
  • whether to show the target in stdout: RUST_LOG_TARGET (don't show it only when set to 0)
  • whether to use ANSI formatting and colors: RUST_LOG_STYLE (don't use it only if set to never)

Also initialized a Prometheus subscriber on the provided port.
Tested on the Hera binary.

closes #33

@merklefruit merklefruit added A-hera Area: OP Stack Rollup node binary A-observability Tracing, metrics, logs and other observability tools labels Aug 27, 2024
@merklefruit merklefruit self-assigned this Aug 27, 2024
@merklefruit merklefruit requested a review from refcell August 27, 2024 20:22
Copy link
Collaborator

@refcell refcell left a comment

Choose a reason for hiding this comment

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

Nice, like the use of RUST_LOG_TARGET

@refcell refcell merged commit b0e797d into main Aug 27, 2024
11 checks passed
@refcell refcell deleted the nico/feat/telemetry branch August 27, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-hera Area: OP Stack Rollup node binary A-observability Tracing, metrics, logs and other observability tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: telemetry stack
2 participants