Bump op-node to v1.19.7 and op-reth to v2.4.4 (PLAT-794) - #142
Conversation
Pin op-node to v1.19.7 and op-reth to v2.4.4 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 7e167ae. Both upstream releases are non-breaking. op-node v1.19.7 is optional and carries no runtime change (test-only helpers dropped from production builds). op-reth v2.4.4 is a recommended release adding canonical-head metrics (reth_chain_head_basefee / _gas_used / _blob_gas_used); these surface on the existing --metrics endpoint with no config change. Verified for this bump: - op-node-lisk-sepolia.patch still applies; system_config.go is byte-identical between v1.19.6 and v1.19.7 (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 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (2)
Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe Docker build now uses op-node v1.19.7 and op-reth v2.4.4, both pinned to commit ChangesVersion pin updates
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to The dependency and documentation updates are consistent and introduce no identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
A rabbit checks the version trail Comment |
What was the problem?
This PR resolves PLAT-794.
op-node and op-reth were pinned to v1.19.6 / v2.4.3. Upstream cut op-node/v1.19.7 (optional — production-build dependency cleanup, no runtime change) and op-reth/v2.4.4 (recommended — observability only, adds canonical-head basefee/gas/blob-gas metrics restoring op-geth-equivalent visibility).
Neither release is breaking, and neither needs an env, peer, entrypoint or flag change.
How was it solved?
reth/Dockerfile— op-nodeVERSION→v1.19.7, op-rethVERSION→op-reth/v2.4.4, bothCOMMIT→7e167ae.README.md— client versions and the op-reth source-tree link.Upstream cut both releases from the same monorepo commit, so the two
COMMITpins share7e167ae. That value is the peeled commit (refs/tags/<tag>^{}), not the annotated tag object — pinning the tag SHA is what broke the build in PLAT-788, since the Dockerfile assertsgit rev-parse HEADequalsCOMMIT.The new op-reth gauges are served on the existing metrics endpoint (host
7301) with no config change. They only advance once the node follows the chain over the Engine API, so they stay flat during snapshot restore and backfill — worth knowing before treating a flat series as a fault.How was it tested?
CI builds the image on linux/amd64 and linux/arm64, which exercises both clone-and-verify steps and both compilations.
Verified against upstream before pinning:
git ls-remote ^{})7e167aeop-node-lisk-sepolia.patchsystem_config.gobyte-identical across v1.19.6/v1.19.7 (blob7b368b2)dockerfile-lisk-sepolia.patchgit apply --checkpasses (sepolia CI path)GOLANG_VERSION=1.26go.mod(go 1.26.0)RUST_VERSION=1.95rust/rust-toolchain.toml(channel = "1.95")superchain-registrysubmodule08d6a44—lisk/lisk-sepoliaresolution untouchedRelease draft v0.4.29 is prepared against
mainand will be published after this merges.Summary by CodeRabbit
Documentation
v1.19.7and op-rethv2.4.4.Chores
v1.19.7and op-rethv2.4.4.