go-ethereum icon indicating copy to clipboard operation
go-ethereum copied to clipboard

Remaining Hive Cancun Test Failures

Open spencer-tb opened this issue 1 year ago • 2 comments

Description

I'm tagging the last 5 failures here for the hive cancun tests. Specifically from the engine-cancun simulator.

(note: on hivecancun all other failures pass locally and are due to something internal to the CI).

ForkchoiceUpdatedV3 To Request Shanghai Payload, Null Beacon Root
ForkchoiceUpdatedV2 To Request Shanghai Payload, Non-Null Beacon Root
ForkchoiceUpdatedV2 To Request Cancun Payload, Non-Null Beacon Root
Invalid PayloadAttributes, Missing BeaconRoot, Syncing=False
Invalid PayloadAttributes, Missing BeaconRoot, Syncing=True

These can all be explained from the execution apis spec update in https://github.com/ethereum/execution-apis/pull/498.

For the first 4 tests:

ForkchoiceUpdatedV3 To Request Shanghai Payload, Null Beacon Root
ForkchoiceUpdatedV2 To Request Shanghai Payload, Non-Null Beacon Root
ForkchoiceUpdatedV2 To Request Cancun Payload, Non-Null Beacon Root
Invalid PayloadAttributes, Missing BeaconRoot, Syncing=False

We are essentially sending an fcu with invalid payload attributes and hence expect an error reponse of INVALID_PAYLOAD_ATTRIBUTES. Geth is returning INVALID_PARAMS.

For the 5th test case where we are syncing:

Invalid PayloadAttributes, Missing BeaconRoot, Syncing=True

We expect no error response, as from the execution apis PR above, if a client is syncing no payload attributes validations should be ran. Currently geth returns with INVALID_PARAMS.

Steps to reproduce

Using the latest version of hive run the following command:

./hive --sim ethereum/engine --client go-ethereum --client-file configs/cancun.yaml --docker.output --sim.limit engine-cancun/"<test_name>"

spencer-tb avatar Feb 08 '24 14:02 spencer-tb

Assuming this is fixed ( @lightclient ?)

holiman avatar Apr 24 '24 07:04 holiman

Sorry these are not resolved yet.

lightclient avatar Apr 24 '24 20:04 lightclient