AllCoreDevs - Testing CL Breakout #090
Transcript
- Mikhail Kalinin
If there is a time I would like to do a quick update on the Gloas compliance testing at the end of the breakout
- potuz
I’d like to mention the Circuit breaker, but I don’t see anyone from Lighthouse here
- Eitan Seri-Levi
Replying to "I’d like to mention ..." Im here
- potuz
Ahh Eitan is here
- potuz
Replying to "I’d like to mention ..." jinx
- Eitan Seri-Levi
Reacted to "jinx" with 😉
- Justin Traglia
I don’t think we should have a boost factor for p2p bids (positive or negative).
- Eitan Seri-Levi
Reacted to "I don’t think we sho..." with 👍
- Jason Vranek
I think config-wise, the complexity comes with per-buidler p2p bid preferenecs, so killing builder_boost but keeping per-builder min bid keeps the complexity
- Eitan Seri-Levi
Yeah boosting specific builders over p2p seems too complex imo
- Justin Traglia
I might have misunderstood. This is more about global vs specific builders, right?
- Eitan Seri-Levi
If u want a specific builder just connect to them directly?
- Luca | Vero
Others feel indifferent I guess 😄
- Nico Flaig
async discussion was here https://discord.com/channels/595666850260713488/1532150870932852857
- Jason Vranek
ah my bad, so global builder boost works
- Jason Vranek
does anyone want per-builder p2p bids? maybe easier question
- Eitan Seri-Levi
Replying to "does anyone want per..." I do not want this
- potuz
I defer my vote to Nico and Terence
- Nico Flaig
Replying to "does anyone want per..." yeeet them
- Jason Vranek
will try to get that update out today
- james
Reacted to "will try to get that update out today" with 👍
- Jason Vranek
it would be localized to #88 keymanagre / 630 beacon api, the builder spec doesnt change
- james
Reacted to "yeeet them" with 👌
- potuz
Sorry missed the last 5 minutes due to a fire alarm
- Justin Traglia
- Nico Flaig
lodestar will still use the right state :D
- potuz
Still having stupid fire alarms
- Justin Traglia
- Justin Traglia
Reacted to "Still having stupid ..." with 🔥
- Jason Vranek
got to run to another call, will get the PRs updated today 🫡
- Justin Traglia
Replying to "got to run to anothe..." Thanks Jason!
- Mikhail Kalinin
Let’s do the compliance testing update next time. There is no rush on that
- NC
Reacted to "Let’s do the complia..." with 👍
Call summary
Decisions
- •Per-builder P2P bid boosting and min-bid config removed from ePBS specs - 00:09:28
- •Builder bid balance-check kept as IGNORE (not REJECT) on gossip path - 00:16:07
- •ePBS API PRs to be implemented by clients before merging; merge after successful devnet validation - 00:11:46
Highlights
- Epbs Builder Api:
- ·Three open PRs: Key Manager API, Beacon API produce-block endpoint, builder spec (PR #165) - 00:01:10
- ·Per-builder P2P bid boosting/min-bid is technically broken — P2P propagates on raw value, not boosted factor - 00:04:02
- ·Consensus to remove per-builder P2P boost/min-bid config; global P2P boost factor retained - 00:09:06
- Epbs Epoch Boundary:
- ·Lodestar seeing double epoch transitions from bids built on parent at epoch boundary; Nico added a guard in Lodestar (PR #9756) - 00:16:28
- ·Potuz's spec PR removes epoch-boundary shuffling-stability check — Lighthouse and Prysm already behave this way on mainnet - 00:17:32
- Epbs Gossip Validation:
- ·Potuz: bid validation against non-head state should remain IGNORE, not REJECT — changing to REJECT would cause Prysm to drop all non-head bids, killing multi-bid functionality - 00:13:04
- ·Consensus to keep balance-check as IGNORE; Lodestar will still use correct state locally - 00:16:07
- Sparse Blob Pool Engine Api:
- ·Bosul: sparse blob pool Engine API discussed at last ACD; Lighthouse raised design concerns but no counter-proposal submitted; other clients already implemented; proceed unless significant objections raised async - 00:19:35
Action Items
- •Jason Vranek - Remove per-builder P2P boost/min-bid config from Key Manager API and Beacon API PRs - 00:09:28
- •Potuz - Add rationale paragraph to epoch-boundary shuffling-stability spec PR - 00:19:01
- •CL teams (esp. Lighthouse) - Review sparse blob pool Engine API proposal async; share objections if any - 00:19:35
Key decisions
Per-builder P2P bid boosting and min-bid config removed from ePBS specs
Consensus to remove per-builder P2P boost/min-bid config; global P2P boost factor retainedBuilder bid balance-check kept as IGNORE (not REJECT) on gossip path
Consensus to keep balance-check as IGNORE; Lodestar will still use correct state locallyePBS API PRs to be implemented by clients before merging; merge after successful devnet validation
Consensus to remove per-builder P2P boost/min-bid config; global P2P boost factor retained