AllCoreDevs — Execution #235
Transcript
Ask about this call
Answered from this call's summary and transcript only.
AI-generated — verify against the recording.
Call summary
Targets
- •Tomorrow - Glamsterdam DevNet 0 launch (Prysm + Geth) — 00:14:50
Decisions
- •Engine API PR 786 selected for implementation (reorg to finalized hash) — 00:38:23
- •EIP-8237 (execution witness) deferred to Hegota, not emergency inclusion — 00:42:20
- •EIP-8163 (0xAE opcode reservation) marked as PFI for Hegota — 01:14:52
- •EIP-7979 (CALLSUB/RETURNSUB) marked as PFI for Hegota — 01:17:33
Highlights
- Client Updates:
- ·Geth JSON marshalling 10x performance improvement PR available — 01:38:08
- Organizational:
- ·Nixo joins as co-lead of All Core Devs alongside Ansgar — 00:09:47
- Eip Discussions:
- ·EIP-8237 (execution witness optimization) proposed; deferred to Hegota — 00:39:08
- ·EIP-8037 dynamic pricing identified as main testing complexity; simplification needed — 00:53:31
- ·Strong concerns about EIP-8037 complexity vs benefits; may need simplification — 01:04:27
- Testing Progress:
- ·BAL DevNet 4 test release consumed by clients; implementation ongoing — 00:10:54
- ·Glamsterdam DevNet 0 launch targeting tomorrow with Prysm and Geth — 00:14:20
- ·EIP-8037 broke ~30% of tests; complexity concerns raised by testing team — 00:47:12
- Fork Status And Schedule:
- ·Engine API PR 786 selected: reorg support up to finalized, not capped — 00:38:23
- ·EIP-7979 (CALLSUB/RETURNSUB) proposed for inclusion in Hegota — 01:14:38
- ·EIP-8163 proposed for inclusion: reserves 0xAE opcode for L2/alt-L1 use — 01:09:32
Action Items
- •EL client teams — Review and provide feedback on Engine API PRs 770/786 — 00:17:26
- •EIP authors and interested teams — Explore simplifications to EIP-8037 before next call — 00:56:17
- •Greg Colvin — Schedule breakout call for EIP-7979 discussion — 01:17:00
Meeting chat312
- nixo
gm
- Barnabas
mg
- Pooja Ranjan
gm
- read.ai meeting notes
Red added read.ai meeting notes to the meeting. Read provides AI generated meeting summaries to make meetings more effective and efficient. View our Privacy Policy at https://www.read.ai/pp Type "read stop" to disable, or "opt out" to delete meeting data.
- nixo
Reacted to "mg" with 🙃
- Barnabas
2 times in a row Justin is just quiet when he joins 😢
- nixo
Replying to "mg" ƃɯ
- Parithosh Jayanthi
Reacted to "2 times in a row Justin is just quiet when he joins 😢" with 🥲
- Josh Davis
Reacted to "2 times in a row Jus..." with 🐻
- Justin Florentine (Besu)
the vibes remain off
- nixo
Replying to "2 times in a row Jus..." did we do something wrong
- jochem-brouwer (Fairphone)
Gm
- nixo
Replying to "the vibes remain off" the silence will stay until the morale improves?
- Parithosh Jayanthi
Reacted to "the silence will stay until the morale improves?" with ❤️
- Justin Florentine (Besu)
Replying to "2 times in a row Jus..." i think there is a reckoning going on.
- Barnabas
Potuz will be 10min late. So maybe we can circle back to his topics later.
- Ansgar Dietrichs
- nixo
episode 😂
- Parithosh Jayanthi
Reacted to "Potuz will be 10min late. So maybe we can circle back to his topics later." with 👍
- Josh Davis
Reacted to "episode 😂" with 🍿
- Milos
Reacted to "episode 😂" with 🍿
- kevaundray wedderburn
Reacted to "episode 😂" with 🍿
- Stefan Starflinger
Reacted to "episode 😂" with 🍿
- Josh Davis
Hooray!!
- Barnabas
Replying to "episode 😂" its soap oepra
- felix (eest)
Reacted to "episode 😂" with 🍿
- Pooja Ranjan
Congratulations, Nixo!
- marc | wolovim
Reacted to "episode 😂" with 🍿
- Tim Beiko
Reacted to "episode 😂" with 🍿
- Parithosh Jayanthi
Congrats! <3
- Barnabas
Reacted to "Congratulations, Nix..." with 👍
- Christine Kim
lets goooo nixo!
- nixo
Reacted to "Congratulations, Nixo!" with 💙
- David (dionysuz)
gz 🍺
- Tim Beiko
Congrats Nixo 🫡
- nixo
Reacted to "Congrats! <3" with 💙
- nixo
Reacted to "lets goooo nixo!" with 💙
- nixo
Reacted to "Congrats Nixo 🫡" with 💙
- nixo
Reacted to "gz 🍺" with 💙
- Mario Havel
Reacted to "episode 😂" with 🍿
- Justin Florentine (Besu)
Replying to "the vibes remain off" it's a good question tbh. the enthusiasm is usually warranted, but lately... I promise it will always be sincere
- Carson
Yaaay nixo!!
- Mario Havel
lfg Nixo!
- nixo
Reacted to "it's a good question tbh. the enthusiasm is usually warranted, but lately... I promise it will always be sincere" with 🫡
- Trent Van Epps
Reacted to "it's a good question..." with 🫡
- Barnabas
Reacted to "it's a good question..." with 🫡
- nixo
Reacted to "Yaaay nixo!!" with 💙
- nixo
Reacted to "lfg Nixo!" with 💙
- Parithosh Jayanthi
Reacted to "it's a good question tbh. the enthusiasm is usually warranted, but lately... I promise it will always be sincere" with 🫡
- marc | wolovim
Reacted to "lfg Nixo!" with 💙
- nixo
Reacted to "Hooray!!" with 💙
- Łukasz Rozmej
I think we did
- Daniel Lehrner (Besu)
Besu: Yes, no issues
- Barnabas
geth now has bal-devnet-4 branch any other client has a branch prepared already?
- FLCL
tomorrow
- Daniel Lehrner (Besu)
Replying to "geth now has bal-devnet-4 branch any other client has a branch prepared already?" We have one as well. But 1 PR still needs to be merged in
- Marius van der Wijden
The new 7 changes did turn the implementation quite ugly for us
- Iván | ethrex
No issues on the ethrex side
- Barnabas
https://notes.ethereum.org/@ethpandaops/glamsterdam-devnet-0
- Stefan Starflinger
bal-devnet-4 spec https://notes.ethereum.org/@ethpandaops/bal-devnet-4
- Barnabas
https://github.com/ethereum/execution-apis/pull/786 or https://github.com/ethereum/execution-apis/pull/770
- Marius van der Wijden
Both are fine, I don't care either way
- Barnabas
Replying to "https://github.com/e..." could EL teams chime in which one would they prefer
- Marius van der Wijden
Replying to "Both are fine, I d..." 786 is more principled, 770 is slightly easier to implement
- Stefan Starflinger
https://github.com/ethereum/execution-apis/pull/770 or https://github.com/ethereum/execution-apis/pull/786
- Barnabas
its been in discussion for over a month. Would prefer to make a decision
- Potuz
I prefer 786 because it is formally correct at least for consensus
- Potuz
+1 for what Marius says, just ship whatever
- Toni Wahrstätter
It feels like, in case there's no finalization for more than 128 blocks, then ELs should keep the state diff / state snapshot available to be able to do auch a deep reorg without falling-back to snap sync. I think Marek mentioned that option last time.
- Potuz
flip a coin
- Marius van der Wijden
128 is also good for us
- Marius van der Wijden
going lower than 128 requires more work
- Toni Wahrstätter
What happens if we hit that cap?
- Potuz
we stall
- Potuz
yeah cause no block can be acccepted
- Marius van der Wijden
Reacted to "yeah cause no bloc..." with 👍
- Potuz
ah that was a reply to Marius
- Francesco Risitano
I believe reth already supports reorg up to finalised using config on the engine
- prestwich
what does “reorg within the canonical chain” mean?
- Potuz
Replying to "what does “reorg wit..." the CL chain has more nuance about what's canonical than the EL, we can reorg to a previous hash of the chain as canonical, but the CL takes a different CL block for it
- prestwich
Reacted to "the CL chain has mor..." with 🙇♀️
- Marius van der Wijden
Replying to "what does “reorg ..." Reorg within the chain that leads to our current head
- Marius van der Wijden
Replying to "what does “reorg ..." Thats our currently canonical chain
- Tomás Arjovsky | Lambda | Ethrex
Is this really a reorg then or is it just building on top of a previous block in the canonical chain?
- prestwich
Replying to "what does “reorg wit..." you’re saying that we may maintain a header chain, but the contents may change?
- Marius van der Wijden
Sounds like we should do the principled solution
- Marius van der Wijden
@toni we have state for the last 90k blocks
- Potuz
we've already wasted enough time on this, the cap of 32 is fine too, it doesn't matter, we will need a majority attacker to trigger this and we'd be already in trouble anyway
- Marius van der Wijden
Reacted to "we've already wast..." with 👍
- Potuz
we will never reorg more than 1 block
- Potuz
ever never
- jochem-brouwer (Fairphone)
We can add this reorg scenario to benchmark scenarios. (In fact we should)
- Barnabas
786 lets go
- Barnabas
Reacted to "786 lets go" with 🚢
- Ahmad Bitar | Nethermind
Reacted to we've already wasted... with "👍"
- kevaundray wedderburn
Reacted to "786 lets go" with 🚢
- Toni Wahrstätter
Replying to "we've already wasted..." ...or a bug
- Louis
Replying to "We can add this reor..." Add in todo list
- Toni Wahrstätter
Replying to "we've already wasted..." Imo such a big attacker is less likely than a bug. At least empirically
- Łukasz Rozmej
FCU->finalized_hash->block_number lookup -> keep one pointer to it
- Potuz
just put the cap internally
- Francesco Risitano
Reacted to "FCU->finalized_hash-..." with 👍
- FLCL
do we need finalized-1?
- jochem-brouwer (Fairphone)
Replying to "We can add this reor..." Thanks 🙏
- Potuz
let's just test depth 1 :) that's all that will happen ever
- Barnabas
Replying to "786 lets go" who removed my own ship? 😂
- kevaundray wedderburn
I think we would need to make an assumption around how long we can have periods of non finality?
- Tomás Arjovsky | Lambda | Ethrex
Replying to "let's just test dept..." This is the only real epbs need, right? The rest is just making a good spec
- Toni Wahrstätter
I'm fine with it but let's just be intentional that ELs would stall if such a scenario ever happens.
- Potuz
Replying to "let's just test dept..." yes unless there is a majority attacker on the validator set that wants to trigger this
- Potuz
Replying to "let's just test dept..." or a bug on clients
- Marius van der Wijden
Replying to "I'm fine with it b..." It wouldn't stall, it would just reject this
- Yann Vonlanthen
Replying to "let's just test dept..." Or asynchrony / network issue
- Potuz
Replying to "let's just test dept..." no that can't be the case for this
- Potuz
Replying to "let's just test dept..." because it would be pretty weird that the CL p2p is gossiping blocks+payloads and not attestations
- Marius van der Wijden
768 and stop arguing
- Barnabas
Reacted to "768 and stop arguing" with 👍
- Barnabas
Reacted to "768 and stop arguing" with 🚢
- Parithosh Jayanthi
Reacted to "768 and stop arguing" with 🚢
- Potuz
Replying to "768 and stop arguing" that's a new one?
- Tomás Arjovsky | Lambda | Ethrex
Agree with 786
- Potuz
Replying to "768 and stop arguing" up to genesis
- Barnabas
Replying to "768 and stop arguing" 786
- nixo
no u right
- Tomás Arjovsky | Lambda | Ethrex
Reacted to "up to genesis" with 😂
- Stefan Starflinger
Reacted to "768 and stop arguing" with 🚢
- Stefan Starflinger
Reacted to "768 and stop arguing" with 👍
- Yann Vonlanthen
Reacted to "because it would be ..." with 👀
- nixo
Reacted to "Screenshot2026_04_23_163146.jpg" with 😂
- FLCL
Reacted to "Screenshot2026_04_23_163146.jpg" with 😂
- Marius van der Wijden
Replying to "768 and stop argui..." I'm not good with numbers
- Bosul Mun
Felix(Geth): Is 786 in now ??
- emma (reth)
Reacted to "episode 😂" with 🍿
- Barnabas
Reacted to "Is 786 in now ??" with 👍
- Marius van der Wijden
Reacted to "Is 786 in now ??" with 👍
- nixo
Reacted to "Is 786 in now ??" with 👍
- Barnabas
Replying to "Felix(Geth): Is 786 ..." please close 770
- Justin Florentine (Besu)
is it useful to CL clients to give them our max reorg depth via a config api?
- Ansgar Dietrichs
sorry for saying “emergency inclusion” - that’s obviously silly in this context, I just meant inclusion after the CFI window has been closed
- Mario Vega
Reacted to "768 and stop arguing" with 🚢
- Dustin
Replying to "is it useful to CL..." for Nimbus CL not really. either it needs the reorg or no
- Barnabas
This was brought up many times in earlier calls, its just got its own EIP now
- Dustin
Replying to "is it useful to CL..." it's going to follow whatever its fork choice requires
- Potuz
again I do not really feel strongly about this, it's a good to have it EL devs think its easy
- Dragan Rakita
Reacted to "Screenshot2026_04_23_163146.jpg" with 😂
- Barnabas
I think I might have misunderstand terence’s comment yesterday, thought it was high urgency for gloas
- Potuz
all I'm asking is for EL devs to take a look at it cause it would really save **a lot** of bwidth and speed to sync clients
- Marius van der Wijden
Why do we need the accumulator?
- Enrico Del Fante (tbenr)
Ye it would be very nice
- Potuz
Replying to "Why do we need the a..." because the attacker can lie what the withdrawals were
- Toni Wahrstätter
Yeah it's def nice and should be done in hegota
- Mario Vega
https://github.com/ethereum/pm/issues/2015#issuecomment-4305210461
- Potuz
Replying to "Why do we need the a..." the validators can be malicious and finalize an invalid block
- emma (reth)
Reacted to "Screenshot2026_04_23_163146.jpg" with 😂
- Potuz
One last thing on 8237 (or against 8237 lol) it's absolutely fine to ship in Hegota cause there's very little tech debt, a couple of structure changes but the whole range requests can be deleted from clients and deprecated immediately as soon as the EIP is SFId, so no problem to include it later
- Marius van der Wijden
Replying to "Why do we need the..." Hmm I see
- Marius van der Wijden
I have to run, I think we need 8037. Otherwise we can't really increase the gas limit
- Marius van der Wijden
I agree that the complexity increased quite significantly
- FLCL
So we agree to try to simplify? Want to try some ideas
- Anders Elowsson
Adapting the price with the gas limit is a “nice-to-have” that increases the probability of setting an “appropriate gas price” (facilitating multidimensional metering with high throughput) at any given gas limit. However, it does not guarantee that an appropriate price is set, because the price will not adapt with actual demand anyway (as in the EIP-8075 proposal). So the best way to understand the functionality of adapting the price with the gas limit is that it makes it more likely that we can set an appropriate gas price across a wider range of gas limits, but it never actually provides any guarantees that the price would be set correctly at any limit. The direct outcome is that a fixed price must be set such that there will be a higher probability that the price is a too high initially, and too low when closing in on the Hegota fork.
- Peter Miller
STEEL has spent all day trying to simply 8037. We came up with approaches to attack all the problems, but the changes were so numerous it was basically a rewrite.
- DanielVF
Reacted to "STEEL has spent al..." with 👍
- pk910
I've been preparing spamoor to work under the new eip 8037 gas model. While doing so, we'Ve noticed that a lot of the deployments take significantly more gas, pushing some of the existing contracts, especially related to Uniswap over the transaction gas limit of 16M. So, implementing 8037 leads to the situation where existing big projects are likely no longer able to deploy their contracts on chain, because the combination of increased state cost and the per tx gas limit leads to a effective reduction of the deployable contract size.
- FLCL
do you have the list of all problems?
- Dragan Rakita
This should be ~fixed~ slightly mitigated with 0 -> x -> 0 rule that is going in devnet4
- Ansgar Dietrichs
@Marius Van Der Wijden (M) do you still have your hand up or again?
- Marius Van Der Wijden (M)
Replying to "I've been preparing ..." Why? We remove the 16m gas limit, so they should be able to deploy
- Barnabas
Replying to "I've been preparing ..." wait we removed the 16m gas limit?
- pk910
Replying to "I've been preparing ..." yea, didn't know that we're removing the per-tx limit too.
- Marius Van Der Wijden (M)
Replying to "I've been preparing ..." Only for regular gas
- Peter Miller
Replying to "I've been preparing ..." There is mechanism that exempts state gas from the cap
- Stefan Starflinger
Reacted to "Lets discuss on sval..." with 👍
- Guillaume
Replying to "Lets discuss on sval..." good job sharing the secret location
- Dragan Rakita
Replying to "I've been preparing ..." Yeah, gas more than 16M can only be spend on state increase.
- Potuz
Reacted to "good job sharing the..." with 🤣
- Stefan Starflinger
Replying to "Lets discuss on sval..." Its in norway
- felix (eest)
Reacted to "good job sharing t..." with 🤣
- pk910
Reacted to "good job sharing the..." with 🤣
- Stefan Starflinger
Reacted to "good job sharing the..." with 🤣
- Maria Silva
Reacted to "do you have the list..." with ➕
- spencer
Reacted to "good job sharing t..." with 🤣
- Louis
Reacted to "good job sharing the..." with 🤣
- Dragan Rakita
Reacted to "good job sharing the..." with 🤣
- Maria Silva
Reacted to "good job sharing the..." with 🤣
- Barnabas
Reacted to "good job sharing the..." with 🤣
- Peter Miller
Replying to "STEEL has spent all ..." I don't have a comprehensive list in writing on me atm, sorry. We will get summary out ASAP.
- FLCL
defining cost via header accumulator of state additions? welcome total state difficulty 😄 (and no 2nd dimension)
- Mario Vega
Reacted to "good job sharing the..." with 🤣
- emma (reth)
Reacted to "good job sharing the..." with 🤣
- Mercy Boma Naps-Nkari
Reacted to "good job sharing the..." with 🤣
- Parithosh Jayanthi
Reacted to "good job sharing the secret location" with 🤣
- Tomás Arjovsky | Lambda | Ethrex
I think the main issue here is not devnet-related. The concern about big contract txs hitting the limit is probably the issue I heard that I find concerning
- Barnabas
Is the risk of a major bug in EIP-8037 really worth the benefit? Do we need that much scaling in the next fork right now, or can we ship a simple state-cost bump this fork and give 8037 another cycle for coverage? I'd rather we prioritize shipping EIPs that are well-scoped and well-tested than shipping something that even our testing team is finding “impossible” to cover.
- DanielVF
I think EIP-8037 is fundamentally flawed - it's both far more complexity than the benifits that it actually brings, and is also the wrong path long term for dealing with charging for state growth.
- Anders Elowsson
Here is the research post: https://ethresear.ch/t/failure-modes-in-eip-8037-and-state-gas-scaling/23975 It discusses state-gas parameter only (SGPO) hard forks. There are various levels of complexity that it could be implemented at.
- felix (eest)
Reacted to "I think EIP-8037 i..." with 👍
- Stefan Starflinger
8037 is really not that bad. Splitting regular gas and state gas makes a lot of sense. I agree, that simplification of the EIP is always a win.
- Dragan Rakita
Imo EIP-8037 is nice change and it unlocks bigger block, so it feels impactful to stay.
- DanielVF
That part isn't that bad. It's the dynamic gas pricing
- Dragan Rakita
dynamic gas pricing is strange
- Stefan Starflinger
Replying to "8037 is really not t..." More backwards compatability considerations in the EIP would be great.
- Mario Vega
Reacted to "dynamic gas pricing ..." with 👍🏼
- Mario Vega
Replying to "dynamic gas pricing ..." indeed
- Stefan Starflinger
Reacted to "dynamic gas pricing ..." with 👍🏼
- Stefan Starflinger
Replying to "dynamic gas pricing ..." I agree
- FLCL
if you have state cost increase you can increase tx limit proportionally
- Mario Vega
Replying to "dynamic gas pricing ..." We always have done costant gas number consumed by opcode and then price the gas in wei, this thing is making the gas number consumed variable
- Barnabas
Just imagine how much headache we gonna give to Devs…
- DanielVF
Reacted to "Just imagine how m..." with 👍
- Stefan Starflinger
Replying to "Just imagine how muc..." ibuprofen
- Maria Silva
Reacted to "ibuprofen" with 😂
- Barnabas
Replying to "Just imagine how muc..." ibuprofen for mascot
- marc | wolovim
Reacted to "ibuprofen for mascot" with 💊
- Barnabas
Reacted to "ibuprofen for mascot" with 💊
- bono | Lambda | ethrex
Reacted to "ibuprofen" with 😂
- bono | Lambda | ethrex
Reacted to "ibuprofen for mascot" with 💊
- jochem-brouwer
aextension :D
- Mario Vega
Reacted to "ibuprofen for mascot" with 💊
- Mario Vega
Reacted to "ibuprofen" with 😂
- Parithosh Jayanthi
Reacted to "ibuprofen for mascot" with 💊
- Parithosh Jayanthi
Reacted to "ibuprofen" with 😂
- Anders Elowsson
Note that SGPOs would allow more accurate pricing than the current EIP, and therefore quite possibly more scaling, because devs can adapt things to actually observed demand.
- Piotr
EIP-8163 - https://eips.ethereum.org/EIPS/eip-8163 https://ethereum-magicians.org/t/eip-8163-reserve-extension-0xae-opcode/27756 * Reserve `EXTENSION (0xae)` for non-Ethereum L1 EVM use * Equivalent to `INVALID (0xfe)` on Ethereum L1 EVM * No implementation effort needed :tada:
- Dragan Rakita
Replying to "dynamic gas pricing ..." Yeah I know, I needed to propagate this variable everywhere so it is a new mechanism I needed to make
- Dragan Rakita
Reacted to "ibuprofen for mascot" with 💊
- Dragan Rakita
Reacted to "ibuprofen" with 😂
- lightclient
the problem is that we need L2s to agree on how they will use the opcodes that are blocked off
- DanielVF
Yes 0xae
- jochem-brouwer
- Danno Ferrin
F range is oversubscribed, more proposed opcodes than space left.
- Danno Ferrin
0xef may be better
- jochem-brouwer
Antwoord verzenden naar "0xef may be better" l2 contract cannot start with this byte if it implements the blockoff of 0xef at start of contract
- Piotr
Reacted to "0xef may be better" with 🙃
- jochem-brouwer
Antwoord verzenden naar "0xef may be better" https://eips.ethereum.org/EIPS/eip-3541
- Danno Ferrin
Replying to "0xef may be better" Right, so start with jumpdest/ef-extansuon
- jochem-brouwer
Antwoord verzenden naar "0xef may be better" ah right yes that works
- Ben Adams
- Piotr
Replying to "the problem is tha..." The EIP-8163 sets some minimal constraints (mainly not breaking JUMPDEST analysis), and punts coordination on the actual extensions to be outside of EIP-8163 scope (suggests coordinating via EthMag to start with). (also to be maybe more explicit - it's L2s as well, but alt L1s primarily)
- Łukasz Rozmej
Replying to "https://eips.ethereu..." Nethermind supports this EIP
- Łukasz Rozmej
Replying to "https://eips.ethereu..." but I agree we do need compiler confirmations
- Felix (Geth)
I have a technical question but cannot speak rn. What is the purpose of the validation phase of EIP-7979
- Felix (Geth)
would it be possible to add CALLSUB/RETURNSUB without this validation phase?
- Dustin
that seems a misguided approach
- Dustin
(not Felix's comment, the Vyper idea of not even looking)
- Felix (Geth)
I'm just curious because there is no rationale section about the existence of the validation
- DanielVF
I've used VM's that work like this, and it worked well.
- kevaundray wedderburn
Would be great if solidity/vyper could form an opinion on it — or else it may be implemented in a way that they in retrospect would not like
- jochem-brouwer
Antwoord verzenden naar "Would be great if ..." I'll forward to vyper
- kevaundray wedderburn
Reacted to "I'll forward to vyper" with ❤️
- Csaba
Not in agenda, but adding a reminder here for EL devs to check the Discv5 tracker at https://notes.ethereum.org/IhGB-cc9S5WClyd9KBjAIA Next steps: Start running some of our devnets with Discv4 off Pass Discv5 hive tests Agree on date to change default Discv4 off Turn Discv4 off
- Felix (Geth)
sorry can someone relay my question please
- Felix (Geth)
thanks for answering it Greg
- Felix (Geth)
but can we then remove the validation?
- Felix (Geth)
I still don't get it
- Tomás Arjovsky | Lambda | Ethrex
Replying to "Not in agenda, but a..." I’m pretty sure we support discv5 now, but I’ll ping you afterwards to see if there’s gaps
- Tomás Arjovsky | Lambda | Ethrex
Replying to "Not in agenda, but a..." We already support discv5 by default and pass the hive tests
- marc | wolovim
#evm channel in R&D is where it has been referenced already
- Josh Davis
Reacted to "#evm channel in R&D ..." with ➕
- Csaba
Replying to "Not in agenda, but a..." Yeah, that was my impression.Thanks
- Csaba
Replying to "Not in agenda, but a..." Also, I’m adding a new column to collect CLI flags to turn off discv4. Could be useful for devnets.
- Developer Uche
- Marius Van Der Wijden (M)
I got kicked, what was the ask?
- Felix (Geth)
just a comment, newPayloadWithWitness is not a part of the standard engine API
- Ansgar Dietrichs
Replying to "I got kicked, what w..." https://github.com/ethereum/execution-apis/pull/773 whether any opinions
- Felix (Geth)
but this change is strange because it moves exactly one method to another API
- Felix (Geth)
(another 'transport' I mean)
- Felix (Geth)
we anyway want to migrate the engine API to another transport
- Developer Uche
Replying to "(another 'transport'..." Yeah
- Felix (Geth)
so it's a bit strange to only do one method
- Marius Van Der Wijden (M)
I also think we should rather rmove the whole engine api (and refactor it)
- Łukasz Rozmej
this also plays well with adding SSZ to engine API -> Nethermind works on this
- Barnabas
Replying to "I also think we shou..." this is not reasonable within the timeframe tho
- Tomás Arjovsky | Lambda | Ethrex
The PR linked has a good amount of metrics I think
- Łukasz Rozmej
Replying to "I also think we shou..." why?
- Dustin
Why is this doing SSZ in a random ad hoc way
- Dustin
please no
- Francesco Risitano
Json represents bytes as hex so one char per nibble. This results in 2 x compared to something binary.
- kevaundray wedderburn
To Felix, yeah we could add ssz to all engine api methods and then this proposal’s diff becomes adding a new engine api endpoint
- Tomás Arjovsky | Lambda | Ethrex
I think this proposal is great to do a PoC about ssz for the engine api
- Felix (Geth)
Replying to "To Felix, yeah we ..." my point is more that there is already work underway to redesign the engine API with a different method
- Tomás Arjovsky | Lambda | Ethrex
Instead of going “we should do a full on rehaul of everythin first” doing an off-standard change for a single method to test it is a good approach
- Developer Uche
Reacted to "I think this proposa..." with 👍
- Tomás Arjovsky | Lambda | Ethrex
and based on how this goes we can also think about a full rehaul of all the engine api
- bono | Lambda | ethrex
Reacted to "I think this proposa..." with 👍
- Felix (Geth)
Replying to "Instead of going ..." the thing is, this is not proposed as "try it", it's proposed as a standard
- kevaundray wedderburn
Replying to "To Felix, yeah we could add ssz to all engine api methods and then this proposal’s diff becomes adding a new engine api endpoint" Isn’t this redesign using ssz too?
- Iván | ethrex
Reacted to "I think this proposa..." with 👍
- Tomás Arjovsky | Lambda | Ethrex
If anyone doesn’t like ssz (mind you, still very much used in consensus) it won’t harm to have counterproposals with measurements
- jonny (geth)
https://github.com/ethereum/go-ethereum/pull/33969 PR for speeding up json marshaling in geth
- Potuz
wait I don't buy that JSON will ever be comparable to SSZ
- Felix (Geth)
we all agree that a binary format is preferable
- Francesco Risitano
Json payload size is doubled by json so then transport becomes the overhead.
- Potuz
that's impossible, we need to go RLP -> JSON-> SSZ on the engine
- Potuz
this can never be comparable to RLP -> SSZ
- Dustin
Replying to "wait I don't buy t..." yeah, it's a question of priority. Using a 600ms JSON decode pessimized Geth decoder as reason for SSZ is odd
- Dustin
Replying to "wait I don't buy t..." I support SSZ
- pk910
Replying to "https://github.com/e..." uff, that's a interesting approach :D basically unrolled the whole generic json parsing. I guess you don't wanna do this for the whole engine api
- Tomás Arjovsky | Lambda | Ethrex
Replying to "Instead of going “we..." Well yeah, but how would you “try it” without it being spoken on acde first? I think bringing it here so client teams can implement PoCs is good manners
- Felix (Geth)
nethermind is doing that redesign
- Tomás Arjovsky | Lambda | Ethrex
Replying to "Instead of going “we..." This is not my proposal btw, I just like the idea of testing how it performs
- Gary Schulte
which eth r&d discord channel to follow for this?
- jochem-brouwer
Antwoord verzenden naar "Would be great if ..." Have forwarded
- Francesco Risitano
Replying to "which eth r&d discor..." #l1-zkevm-protocol
- Dustin
Replying to "and based on how t..." The risk it ends up as random compatibility jank after that
- Mercy Boma Naps-Nkari
https://github.com/ethereum/execution-apis/pull/755 https://github.com/ethereum/execution-apis/pull/774
- Gary Schulte
Reacted to "#l1-zkevm-protocol" with 🙏
- jonny (geth)
Replying to "https://github.com/e..." It’s really only an issue for endpoints that could potentially have large payload sizes so I dont think it is necessary to do the whole engine api
- Gary Schulte
Replying to "and based on how thi..." right. the xkcd standards comic
- Ansgar Dietrichs
Not in agenda, but adding a reminder here for EL devs to check the Discv5 tracker at https://notes.ethereum.org/IhGB-cc9S5WClyd9KBjAIA Next steps: Start running some of our devnets with Discv4 off Pass Discv5 hive tests Agree on date to change default Discv4 off Turn Discv4 off
- Ansgar Dietrichs
- Bosul Mun
Reacted to "https://github.com/e..." with ❤️
- Developer Uche
Reacted to "It’s really only an ..." with 👍
Key decisions
Engine API PR 786 selected for implementation (reorg to finalized hash)
headlinerEngine API PR 786 was selected for implementation with a reorg to finalized hashEIP-8237 (execution witness) deferred to Hegota, not emergency inclusion
UnknownEIP-8237 was deferred to Hegota and not included in the emergency inclusionEIP-8163 (0xAE opcode reservation) marked as PFI for Hegota
PFIEIP-8163 was marked as PFI for HegotaEIP-7979 (CALLSUB/RETURNSUB) marked as PFI for Hegota
PFIEIP-7979 was marked as PFI for Hegota

