Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hive PrevRandao OpCode Transactions Test fails #4810

Closed
1 task done
Tracked by #2625
Rjected opened this issue Sep 27, 2023 · 1 comment
Closed
1 task done
Tracked by #2625

hive PrevRandao OpCode Transactions Test fails #4810

Rjected opened this issue Sep 27, 2023 · 1 comment
Labels
A-execution Related to the Execution and EVM C-bug An unexpected or incorrect behavior C-test A change that impacts how or what we test E-cancun Related to the Cancun network upgrade M-prevent-stale Prevents old inactive issues/PRs from being closed due to inactivity S-needs-investigation This issue requires detective work to figure out what's going wrong

Comments

@Rjected
Copy link
Member

Rjected commented Sep 27, 2023

Describe the bug

The hive test PrevRandao Opcode Transactions Test (BlobTransactions) fails with an invalid state root:

// previous logs too big to show due to blobs
>> (237f2bce) {"jsonrpc":"2.0","id":31,"method":"engine_newPayloadV3","params":[{"parentHash":"0x9f199b351f16e6cf3833b3894898badc6ae7b9212dc88dfbfcfa31466628cbdc","feeRecipient":"0x0000000000000000000000000000000000000000","stateRoot":"0xbb20110054ea14b82b749bd1c99be77ed2ba26e0a42108dae3ab756591c3ebf0","receiptsRoot":"0xe610d851a7fc9b06c4eb4472911f6434c4c5f5877400edff4f1e0bec451e24a6","logsBloom":"0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000","prevRandao":"0xae829c733b49214dbb6bc39f79b6f739653e54fda02c20d40df8319579e02f98","blockNumber":"0x8","gasLimit":"0x2fefd8","gasUsed":"0xa860","timestamp":"0x123c","extraData":"0x9a726574682f76302e312e302d616c7068612e31302f6c696e7578","baseFeePerGas":"0x14f7976b","blockHash":"0x14f76f8d73d75625b51654ba631eac93dc9c5f1866f4d16f7d9017bafe23576b","transactions":["0x03f901390780843b9aca008506fc23ac00830124f89400000000000000000000000000000000000003168080c0843b9aca00f8c6a0010657f37554c781402a22917dee2f75def7ab966d7b770905398eba3c444014a001521d528ad0c760354a4f0496776cf14a92fe1fb5d50e959dcea1a489c7c831a001453362c360fdd8832e3539d463e6d64b2ee320ac6a08885df6083644a063e7a001a8a4a49dcd1b91c376c87d7d6a6e73ee3792205864bf61781e8e3ad19d0092a001069693395fb9a698b257e6c25380f32393bc0cf17a290f9e7fcea3c4ae7b8ba001111dafbfcc0caa0803fc22f697fd3b9171252504e14ec0443f04a2b328871580a0655c97d10fa762ed99c21b651e77e2e710b14cf8119c0945a5a78f1961334215a07b6034364723bcbeb6ed6658691b9b665137734469ccc26bf5352994f70c7194"],"withdrawals":[{"index":"0x47","validatorIndex":"0x0","address":"0x0000000000000000000000000000000000000000","amount":"0x64"},{"index":"0x48","validatorIndex":"0x1","address":"0x0100000000000000000000000000000000000000","amount":"0x64"},{"index":"0x49","validatorIndex":"0x2","address":"0x0200000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4a","validatorIndex":"0x3","address":"0x0300000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4b","validatorIndex":"0x4","address":"0x0400000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4c","validatorIndex":"0x5","address":"0x0500000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4d","validatorIndex":"0x6","address":"0x0600000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4e","validatorIndex":"0x7","address":"0x0700000000000000000000000000000000000000","amount":"0x64"},{"index":"0x4f","validatorIndex":"0x8","address":"0x0800000000000000000000000000000000000000","amount":"0x64"},{"index":"0x50","validatorIndex":"0x9","address":"0x0900000000000000000000000000000000000000","amount":"0x64"}],"blobGasUsed":"0xc0000","excessBlobGas":"0x240000"},["0x010657f37554c781402a22917dee2f75def7ab966d7b770905398eba3c444014","0x01521d528ad0c760354a4f0496776cf14a92fe1fb5d50e959dcea1a489c7c831","0x01453362c360fdd8832e3539d463e6d64b2ee320ac6a08885df6083644a063e7","0x01a8a4a49dcd1b91c376c87d7d6a6e73ee3792205864bf61781e8e3ad19d0092","0x01069693395fb9a698b257e6c25380f32393bc0cf17a290f9e7fcea3c4ae7b8b","0x01111dafbfcc0caa0803fc22f697fd3b9171252504e14ec0443f04a2b3288715"],"0x1a43fb03fab31b470decb0bf6d19033f3b565d6995ddfe1cc378b561cbd9eff8"]}
<< (237f2bce) {"jsonrpc":"2.0","result":{"status":"INVALID","latestValidHash":"0x9f199b351f16e6cf3833b3894898badc6ae7b9212dc88dfbfcfa31466628cbdc","validationError":"Block state root (0xb3e5e43b5ba2cde715c8ce27c29ebadc25db15034cc1cc41c9818dcdc9a4794d) is different from expected: (0xbb20110054ea14b82b749bd1c99be77ed2ba26e0a42108dae3ab756591c3ebf0)"},"id":31}
CLMocker: Executed payload on 237f2bce: {INVALID 0x9f199b351f16e6cf3833b3894898badc6ae7b9212dc88dfbfcfa31466628cbdc 0x40004664e0}
CLMocker: BroadcastNewPayload Response (237f2bce): &{INVALID 0x9f199b351f16e6cf3833b3894898badc6ae7b9212dc88dfbfcfa31466628cbdc 0x40004664e0}

I'm not sure why this happens, and have not investigated what this test does.

Steps to reproduce

Run hive:

./hive --sim ethereum/engine --client reth --sim.limit "cancun/PrevRandao OpCode"

Node logs

No response

Platform(s)

No response

What version/commit are you on?

No response

What database version are you on?

No response

If you've built Reth from source, provide the full command you used

No response

Code of Conduct

  • I agree to follow the Code of Conduct
@Rjected Rjected added C-bug An unexpected or incorrect behavior S-needs-investigation This issue requires detective work to figure out what's going wrong A-execution Related to the Execution and EVM C-test A change that impacts how or what we test E-cancun Related to the Cancun network upgrade labels Sep 27, 2023
@Rjected Rjected added the M-prevent-stale Prevents old inactive issues/PRs from being closed due to inactivity label Sep 27, 2023
@Rjected
Copy link
Member Author

Rjected commented Dec 15, 2023

We now pass this

@Rjected Rjected closed this as completed Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-execution Related to the Execution and EVM C-bug An unexpected or incorrect behavior C-test A change that impacts how or what we test E-cancun Related to the Cancun network upgrade M-prevent-stale Prevents old inactive issues/PRs from being closed due to inactivity S-needs-investigation This issue requires detective work to figure out what's going wrong
Projects
Archived in project
Development

No branches or pull requests

1 participant