Repository navigation
Bump op-node to v1.19.9 and op-reth to v2.6.0 (PLAT-850) - #144
Conversation
Pin op-node to v1.19.9 and op-reth to v2.6.0 in reth/Dockerfile; update README client versions and source-tree link. Upstream cut both releases from the same monorepo commit, so the two COMMIT pins share da6d325. op-reth v2.6.0 is recommended for all operators. Its breaking changes are inert here: reth-entrypoint passes none of the removed --engine.sparse-trie-* flags, Lisk has no pre-Regolith blocks (isSystemTx), and eth_getMultiProof was not served by v2.5.0. The engine persistence defaults change (threshold 2 -> 7, 5 in-memory blocks) applies, since the image runs on defaults. op-node v1.19.9 drops the op_node_default_rpc_server_* metrics; its sequencer change doesn't apply. Verified for this bump: - op-node-lisk-sepolia.patch still applies; system_config.go is byte-identical between v1.19.8 and v1.19.9 (blob 7b368b2). - dockerfile-lisk-sepolia.patch still applies. - GOLANG_VERSION=1.26 stays aligned with go.mod (go 1.26.0) and RUST_VERSION=1.95 with rust/rust-toolchain.toml (channel 1.95). - superchain-registry submodule is unchanged at 08d6a44. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe Docker build now pins ChangesOP Stack Version Update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Possibly related PRs
Merge Risk: ⚪ Minimal · up to The upgrade removes some OP Node RPC metric series, so dashboards that depend on them would lose those measurements. No such consumer or other concrete regression is established in the supplied evidence; the PR is mergeable after normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
A rabbit hops past pinned tools, Comment |
Without --tag, just's install.sh resolves the latest release through the unauthenticated GitHub API, which returned 403 on the shared arm64 CI runner and failed the build. Pinning the tag skips that lookup; the archive comes from the release download URL instead. 1.46.0 matches upstream optimism mise.toml at the op-node/op-reth pin (da6d325). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
What was the problem?
This PR resolves PLAT-850.
op-node and op-reth were pinned to v1.19.8 / v2.5.0. Upstream cut op-reth/v2.6.0, recommended for all operators, and op-node/v1.19.9, recommended for sequencers and optional for other nodes.
No env, peer, entrypoint or flag change is needed.
How was it solved?
reth/Dockerfile: op-nodeVERSION→v1.19.9, op-rethVERSION→op-reth/v2.6.0, bothCOMMIT→da6d325(the peeled commit both tags point to).README.md: client versions and the op-reth source-tree link.None of op-reth's breaking changes require a change here. The entrypoint passes none of the removed
--engine.sparse-trie-*flags. TheisSystemTxformat change only affects pre-Regolith deposits, which Lisk doesn't have.eth_getMultiProofwas never served. The new engine persistence defaults do apply, since the image runs on defaults: 5 in-memory blocks, threshold 7. op-node drops theop_node_default_rpc_server_*metrics, which Lisk nodes export on port 7300; the release notes flag this for dashboard owners.How was it tested?
CI builds the image on linux/amd64 and linux/arm64, which runs both clone-and-verify steps and both compilations. I did not build locally.
Evidence (6 checks against upstream)
op-node/v1.19.9andop-reth/v2.6.0both resolve toda6d325op-node-lisk-sepolia.patchgit apply --checkpasses;system_config.gobyte-identical across v1.19.8/v1.19.9 (blob7b368b2)dockerfile-lisk-sepolia.patchgit apply --checkpasses against the new DockerfileGOLANG_VERSION=1.26/RUST_VERSION=1.95go.mod(go 1.26.0) andrust/rust-toolchain.toml(1.95);op-node/Makefile,rust/op-reth/bin/Cargo.tomland[profile.maxperf]unchanged;just build-superchain-gostill presentsuperchain-registrysubmodule08d6a44eth_getMultiProofin v2.5.0crates/rpcof its reth pin (op-rs/reth@aef8d3e); v2.6.0 explicitly unserves it (rust/op-reth/crates/node/src/node.rs)Release draft v0.4.31 is prepared against
mainand will be published after this merges.🤖 Generated with Claude Code
Summary by CodeRabbit
justinstaller is pinned to version 1.46.0.