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 Layer Meeting 159 #754

Closed
timbeiko opened this issue Apr 6, 2023 · 9 comments
Closed

Execution Layer Meeting 159 #754

timbeiko opened this issue Apr 6, 2023 · 9 comments

Comments

@timbeiko
Copy link
Collaborator

timbeiko commented Apr 6, 2023

Meeting Info

Agenda

@ralexstokes
Copy link
Member

ralexstokes commented Apr 10, 2023

I've made substantial updates to EIP-4788 I'd like to present. I strongly suggest we CFI EIP-4788 for Cancun.

ethereum/EIPs#6859

I'd also separately like to discuss a block verification RPC method that is used by mev-boost relays to verify incoming builder blocks. Currently, most of that ecosystem uses a fork of geth and I'd like to discuss cross-client implementation of the RPC method to support client diversity at that part of the stack.

@mkalinin
Copy link
Contributor

I'd like to discuss the following fix to the Engine API spec

@timbeiko
Copy link
Collaborator Author

Added @mkalinin @ralexstokes !

@siladu
Copy link

siladu commented Apr 13, 2023

I'd like to discuss the following fix to the Engine API spec

This besu issue has some details if you want more background context hyperledger/besu#5321

I am pretty sure nimbus-nethermind will have the same issue. I wasn't able to confirm whether erigon had the same fix that geth has.

These calls are too late for me, but I think some others have context now since discussing on #interop

@MarekM25
Copy link

I'd like to discuss the following fix to the Engine API spec

This besu issue has some details if you want more background context hyperledger/besu#5321

I am pretty sure nimbus-nethermind will have the same issue. I wasn't able to confirm whether erigon had the same fix that geth has.

These calls are too late for me, but I think some others have context now since discussing on #interop

That's true. We're generating distinct values with a small exception - withdrawals. I discussed with @g11tech that this situation is not possible to happen. I mean

1. Timestamp = A, PrevRandao = X, SuggestedFee = Y and Withdrawals = B, HeadHash = C
2. Timestamp = A, PrevRandao = X, SuggestedFee = Y and Withdrawals = Z,  HeadHash = C 

However, it sounds like it is possible with Nimbus combos, so we will prepare fix

@parithosh
Copy link
Member

I'd like to bring up the topic of capella genesis on the EL&CL front (I'd assume EL has no problem with it). We'd like to modify all the testing infra to start from capella instead of a bellatrix genesis state. We'd additionally need some input on if the various future upgrade branches also support this, i.e, I know verkle needs a rebase before this is possible, but what about EOF or 4844?

@g11tech
Copy link

g11tech commented Apr 13, 2023

That's true. We're generating distinct values with a small exception - withdrawals. I discussed with @g11tech that this situation is not possible to happen. I mean

1. Timestamp = A, PrevRandao = X, SuggestedFee = Y and Withdrawals = B, HeadHash = C
2. Timestamp = A, PrevRandao = X, SuggestedFee = Y and Withdrawals = Z,  HeadHash = C 

well it should not be possible with any CL since withdrawals is a matter of state unless nimbus is not mapping timestamp with the slot (which for e.g. lodestar does)

I would like to understand when can two fcUs have everything same except the withdrawals.

@timbeiko
Copy link
Collaborator Author

Added, @parithosh

@timbeiko
Copy link
Collaborator Author

Closed in favor of #759

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

No branches or pull requests

7 participants