Skip to content

fix(network): Silver startup/liveness ask its API; arch check reads the node; warn when sync restarts the EL - #904

Merged
OisinKyne merged 1 commit into
mainfrom
fix/silver-rc2
Nov 5, 2026
Merged

OisinKyne merged 1 commit into
mainfrom
fix/silver-rc2

Conversation

@OisinKyne

Copy link
Copy Markdown
Contributor

Summary

Follow-ups from the rc1 runs on the dev box (Hoodi and mainnet Silver).

  • Silver startup and liveness probes are now HTTP. Both call GET /eth/v1/node/version, matching fix(silver): HTTP probes, info logs, amd64 node selector; bump to alpha.12 ethpandaops/ethereum-helm-charts#505. Until then they were tcpSocket probes, which pass as long as the port accepts connections.
  • Silver v0.1.0-alpha.13 ("clustering fixes", clustering fixes gattaca-com/silver#470). The version fix, Report the crate version in the agent version gattaca-com/silver#473, merged after alpha.13 was cut, so it lands in the next alpha.
  • The arch check asks the cluster.
    • network install and network sync used to compare Silver's amd64-only image against the CLI's runtime.GOARCH.
    • They now read the nodes' kubernetes.io/arch label, with a 5 s request timeout. If the cluster doesn't answer, they fall back to runtime.GOARCH.
    • With mixed nodes, a non-amd64 node fails the check, since nothing pins the beacon pod to an amd64 node.
    • Only amd64-only clients trigger the lookup, so other installs make no extra cluster call.
  • network sync warns when it will also restart the EL.
    • Deployments made before obol set per-network memory requests run their EL StatefulSet with none. The next sync adds them, which restarts the EL even when the user only meant to change the CL.
    • The host hit this on mainnet: reth restarted during the lighthouse → Silver switch.
    • Sync now reads the live ethereum-execution StatefulSet. If it has no memory request (and the node isn't stopped), sync prints: This sync also restarts reth (the execution client) once: ….

…he node; warn when sync restarts the EL

- Silver startup and liveness probes are httpGet /eth/v1/node/version
  (as ethpandaops/ethereum-helm-charts#505), so a boot stalled before the
  first engine-API exchange restarts after 30 min, and a hung API after 6.
- Silver v0.1.0-alpha.13.
- The amd64-only check reads the cluster nodes' kubernetes.io/arch label
  when the cluster answers, falling back to the CLI's GOARCH.
- network sync warns when it will also restart the EL: deployments made
  before per-network memory requests get them on their next sync.
@OisinKyne
OisinKyne merged commit d4addc1 into main Nov 5, 2026
12 of 13 checks passed
@OisinKyne
OisinKyne deleted the fix/silver-rc2 branch November 5, 2026 17:48
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.

1 participant