testing: add testing_commitBlockV1 spec, fixtures, and shared gas limit env#801
Open
MysticRyuujin wants to merge 2 commits into
Open
testing: add testing_commitBlockV1 spec, fixtures, and shared gas limit env#801MysticRyuujin wants to merge 2 commits into
MysticRyuujin wants to merge 2 commits into
Conversation
Adds the spec yaml and testgen fixtures for testing_commitBlockV1, the write companion of testing_buildBlockV1. The method builds a block from the provided parameters on top of the current canonical head, inserts it, and sets it as the new head, returning the block hash. It's the read/write counterpart to testing_buildBlockV1, skipping the engine_newPayload + engine_forkchoiceUpdated round-trip used by consensus clients. This refines the original spec proposal in ethereum#787 by dropping the parentHash parameter (matching the Nethermind impl from NethermindEth/nethermind#11385). Also adds HIVE_TARGET_GAS_LIMIT to forkenv.json so geth and Nethermind target the same gas-limit ceiling under hive — without this both clients diverge on the target gas limit and produce non-matching block hashes, which made the testing_buildBlockV1 strict-match tests already on main fail on Nethermind. Pinning to 60000000 (geth's miner default) keeps the recorded fixtures identical across clients. testing_commitBlockV1 mutates the canonical head, so the MethodTests entry is registered after testing_buildBlockV1 and before the txpool tests to match hive's lexical replay order. The from-mempool subtest is named "z-from-mempool" so lexical sort puts it last (its result is SpecOnly because mempool composition is non-deterministic).
This was referenced May 18, 2026
MysticRyuujin
added a commit
to MysticRyuujin/hive
that referenced
this pull request
May 18, 2026
…E_TARGET_GAS_LIMIT Without an explicit target, geth and Nethermind use different default gas-limit policies under the engine API: geth's CalcGasLimit moves the parent gas limit toward miner.gaslimit (default 60M) by 1/1024 per block, while Nethermind keeps the parent gas limit unchanged unless Blocks.TargetBlockGasLimit is set. With the rpc-compat chain head sitting at ~75M, the two clients diverge from block 1 onward, so blocks produced by testing_buildBlockV1 / testing_commitBlockV1 in rpc-compat replay get different hashes between clients and the strict-match tests fail on Nethermind. Wiring a new HIVE_TARGET_GAS_LIMIT env var through geth.sh (--miner.gaslimit) and Nethermind's mkconfig.jq (Blocks.TargetBlockGasLimit) makes both clients target the same ceiling. With the value set to 60000000 in execution-apis/tools/chain/forkenv.json, CalcGasLimit produces the same gas limit at every step on both clients, so the recorded rpc-compat fixtures replay byte-exact on both. Spec and cross-client fixtures: ethereum/execution-apis#801
This was referenced May 18, 2026
LukaszRozmej
pushed a commit
to NethermindEth/nethermind
that referenced
this pull request
May 18, 2026
The spec for testing_commitBlockV1 defines txRlps as oneOf [array, null] where null instructs the client that it MAY build from the local txpool. testing_buildBlockV1 already takes IEnumerable<byte[]>? on this branch, but the commit variant still takes a non-nullable IEnumerable<byte[]>, so JSON-RPC requests with a null txRlps fail at parameter binding with -32602 "Missing parameter does not have a default value" before ever reaching the implementation. ProduceBlockAsync already handles `txRlps is null` correctly (it falls back to env.TxSource.GetTransactions). Making the entry-point parameter nullable on both ITestingRpcModule and TestingRpcModule is enough to make the spec-compliant null case dispatch. Spec and cross-client fixtures: ethereum/execution-apis#801
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
1 participant
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.
Summary
Adds the spec yaml and
make fillfixtures fortesting_commitBlockV1, the write companion oftesting_buildBlockV1. The method builds a block from the supplied attributes on top of the current canonical head, inserts it, and sets it as the new head — the read/write counterpart oftesting_buildBlockV1, skipping theengine_newPayload+engine_forkchoiceUpdatedround-trip used by consensus clients.This refines the original spec proposal in #787 by dropping the
parentHashparameter to match the Nethermind reference implementation in NethermindEth/nethermind#11385. The method always operates on the current canonical head.Companion PRs
This PR is the anchor for cross-client interop. The other three implementations agreeing with this spec ship in:
testing_commitBlockV1ineth/catalysttxRlpsnullable intesting_commitBlockV1HIVE_TARGET_GAS_LIMITthroughgeth.shandmkconfig.jqso both clients target the same gas-limit ceilingWhat's in this PR
src/testing/testing_commitBlockV1.yaml— OpenRPC spec for the new method.tools/testgen/generators.go— five subtests: empty txs, with-transactions, with-extra-data, invalid-transaction, and from-mempool (SpecOnly). Subtest names are lexically ordered to match hive'sfilepath.Walkreplay order so chain-head progression at fill time matches at replay time.tests/testing_commitBlockV1/*.io— recorded fixtures via go-ethereum +make fill. Re-runningmake fillproduces byte-identical output (full corpus diff is empty across consecutive runs).tools/chain/forkenv.json+tests/forkenv.json— addsHIVE_TARGET_GAS_LIMIT=60000000(geth's default--miner.gaslimit). With both clients targeting the same ceiling,CalcGasLimitproduces the same gas limit at each block, so the recorded fixtures replay byte-exact on both. This also rescues the existingtesting_buildBlockV1strict-match tests, which silently diverged on Nethermind before this fix.Why drop
parentHash?The Nethermind reference impl in #11385 takes
(payloadAttributes, txRlps, extraData?)and builds on top of the current head — since the method is committing to the canonical chain, there's no useful semantics for an arbitrary parent. The spec yaml here matches that signature exactly.Test plan
make build(specgen) passesmake test(speccheck) passesmake fillagainst local go-ethereum produces byte-identical output across consecutive runsrpc-compatsimulator passes 216/216 against go-ethereum with the new fixturesrpc-compatsimulator passes 9/9testing_*tests against patched Nethermind (the 7 remaining Nethermind failures are pre-existingeth_simulateV1andtxpool_contentissues unrelated to this work)