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

EXECUTION_PAYLOAD_HEADER_FIELD_TREE_HEIGHT changed from 4->5 #394

Closed
wants to merge 1 commit into from

Conversation

Sidu28
Copy link
Contributor

@Sidu28 Sidu28 commented Jan 19, 2024

As per deneb upgrade, an additional 2 fields are added to the ExecutionPayloadHeader container which means that it merkelizes differently (theres an extra layer to the tree). This PR updates the goerli deployments, it is branched off this commit hash as per the deployment tracking spreadsheet: af8d8755fdad94705479679c7c63d454c4ed9a8e

@ChaoticWalrus
Copy link
Contributor

ChaoticWalrus commented Jan 19, 2024

it is branched off this commit hash as per the deployment tracking spreadsheet: af8d8755fdad94705479679c7c63d454c4ed9a8e

Ahh, I see what happened. That is the commit hash of the proxy deployment. The implementations are all at https://github.com/Layr-Labs/eigenlayer-contracts/tree/1d926f25f3bd401c4d6552f43aa561782982cb00

Note that this also matches the commit of the goerli-m2-deployment tag: https://github.com/Layr-Labs/eigenlayer-contracts/tree/goerli-m2-deployment

As per deneb upgrade, an additional 2 fields are added to the ExecutionPayloadHeader container which means that it merkelizes differently (theres an extra layer to the tree).

OK so just another layer to the tree? No need to add any fields to struct or anything?
If you are confident this is sufficient, then please regen this based off the correct commit, and we can get deployment done.

@Sidu28 Sidu28 closed this Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants