Repository navigation
fix(network): Silver logs at info and is ready only when its beacon API answers - #901
Merged
Merged
Conversation
…PI answers Silver logs only errors without RUST_LOG, and the chart's tcpSocket readiness probe reported it ready while its API answered nothing.
Silver publishes linux/amd64 images only, and the k3d node runs the Docker host's architecture. Install and sync now stop with a message naming the other clients instead of leaving a pod that never starts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix(network): Silver: info logs, HTTP readiness, refuse non-amd64 hosts
Found on the v0.18.0-rc0 lighthouse → Silver switch on Hoodi:
Logs: Silver logs only errors unless
RUST_LOGis set. Its first run was silent for 7 minutes, and a later9-minute run produced one line. Set
RUST_LOG=info.Readiness: the ethpandaops chart's probes are
tcpSocket. Silver's API accepted connections but answered nothingwhile it waited on a checkpoint, and the pod still showed
1/1 Ready, so theethereum-beaconService kept routingto it.
GET /eth/v1/node/health, which returns 206 while syncing and counts as ready.tcpSocket: nullreplaces the chart default rather than merging into it.
Architecture: Silver publishes linux/amd64 images only, and the k3d node runs the Docker host's architecture.
obol network install ethereum --consensus-client silverandobol network syncnow refuse on any other host(e.g. Apple Silicon). The message names the other clients. Sync checks too, because
values.yamlcan be edited by hand.Only applies when
consensusClient: silver. A rendered lighthouse deployment has neither change.Tested with
go test ./internal/embed/... ./internal/network/...(including acheckClientArchtable test),golangci-lint --new-from-revclean,, plushelmfile templatefor silver (env and probeas intended) and for lighthouse (unchanged).
🤖 Generated with Claude Code