Skip to content

feat: log client version on startup#87

Merged
MegaRedHand merged 6 commits intomainfrom
log-version-at-startup
Feb 4, 2026
Merged

feat: log client version on startup#87
MegaRedHand merged 6 commits intomainfrom
log-version-at-startup

Conversation

@MegaRedHand
Copy link
Collaborator

@MegaRedHand MegaRedHand commented Feb 2, 2026

This PR adds a log at startup that includes the client version. It also adds a parameter to the docker-build Makefile target to select the docker tag to use.

It also fixes an issue with the Dockerfile that caused version output to be ethlambda ethlambda/v0.1.0-VERGEN_IDEMPOTENT_OUTPUT-VERGEN_IDEMPOTENT_OUTPUT/aarch64-unknown-linux-gnu/rustc-v1.92.0.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 2, 2026

Greptile Overview

Greptile Summary

Adds an explicit startup log line that includes the full CLIENT_VERSION string, so operators can see the exact build (pkg version + git branch/SHA + target triple + rustc) immediately on launch.

This is consistent with existing usage of version::CLIENT_VERSION for the CLI --version flag and for node info metrics (set_node_info).

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • The change is a single additional tracing::info! event emitted after the subscriber is installed; it uses an existing compile-time constant already relied upon for CLI versioning and metrics, and does not alter control flow or data handling.
  • No files require special attention

Important Files Changed

Filename Overview
bin/ethlambda/src/main.rs Logs the client version at startup via a new tracing::info! event.

Sequence Diagram

sequenceDiagram
  participant User
  participant Main as bin/ethlambda/src/main.rs::main
  participant Tracing as tracing subscriber

  User->>Main: start `ethlambda`
  Main->>Tracing: set_global_default(subscriber)
  Main->>Main: CliOptions::parse()
  Main->>Main: print ASCII_ART
  Main->>Tracing: info!(version=CLIENT_VERSION, "Starting ethlambda")
  Main->>Tracing: info!("got node key")
  Main->>Main: initialize genesis/store/blockchain
  Main->>Main: spawn P2P task
  Main->>Main: start RPC server
  Main->>Tracing: info!("Node initialized")
Loading

@MegaRedHand MegaRedHand merged commit 7dbfa13 into main Feb 4, 2026
2 checks passed
@MegaRedHand MegaRedHand deleted the log-version-at-startup branch February 4, 2026 19:03
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.

2 participants