FOCIL Breakout #039
Transcript
- Mehdi Aouadi
gm gn
- Justin Traglia
Reacted to "gm gn" with 😄
- Nico Flaig
we can interop with teku but we are on devnet-8 now :D
- Mehdi Aouadi
Reacted to "we can interop with teku but we are on devnet-8 now :D" with 🙈
- Mehdi Aouadi
Replying to "we can interop with ..." we don’t have a devnet-8 branch yet 😄
- Jihoon
- Nico Flaig
Replying to "we can interop with ..." our trunk branch is on devnet-8 so rebasing focil against devnet-7 is hard now and probably not worth it
- Jihoon
- Justin Traglia
Reacted to "our trunk branch is ..." with 👍
- Jihoon
- Jihoon
- Marc
do any of these need EL changes?
- Mehdi Aouadi
Reacted to "our trunk branch is on devnet-8 so rebasing focil against devnet-7 is hard now and probably not worth it" with 💯
- Jihoon
- Mehdi Aouadi
Replying to "we can interop with ..." I’ll rebase our focil branch once we have devnet-8 branch out
- Nico Flaig
Reacted to "I’ll rebase our foci..." with 👍
- Nico Flaig
(slot, dependent_root) is how we cache proposer preferences
- Nico Flaig
private readonly bySlot = new Map<Slot, Map<RootHex, gloas.SignedProposerPreferences>>();
- Nico Flaig
prune(currentSlot: Slot): void { for (const slot of this.bySlot.keys()) { if (slot < currentSlot) this.bySlot.delete(slot); } }
- Nico Flaig
- Jihoon
Reacted to "for ref https://github.com/ChainSafe/lodestar/blob/unstable/packages/beacon-node/src/chain/opPools/proposerPreferencesPool.ts" with 👍
- Mehdi Aouadi
in Teku we do (Bytes32 dependentRoot, UInt64 proposalSlot, UInt64 validatorIndex) for proposer preferences
- Barnabas
on top of alpha 13 pls
- Jihoon
Reacted to "in Teku we do (Bytes32 dependentRoot, UInt64 proposalSlot, UInt64 validatorIndex) for proposer preferences" with 👍
- Mehdi Aouadi
Replying to "in Teku we do (Bytes..." https://github.com/Consensys/teku/blob/a6dd6502d73c740c6f3fd30db35da7fbb230fc57/ethereum/statetransition/src/main/java/tech/pegasys/teku/statetransition/validation/ProposerPreferencesGossipValidator.java#L232
Call summary
Targets
- •FOCIL-only devnet — based on Alpha 13+ spec; timeline TBD - 00:23:43
Decisions
- •Cut a new FOCIL spec release beyond Alpha 13 to capture recent changes before FOCIL-only devnet - 00:24:34
Highlights
- Spec Updates:
- ·Engine API PR merged; constant renamed from max_bytes_per_inclusion_list to max_transaction_bytes_per_inclusion_list for clarity - 00:09:04
- ·PR #5487: IL store now stores signed inclusion lists to support request/response - 00:10:37
- ·PR #5513 (open): replaces IL committee root with dependent root in IL; adds basic sanity checks — review requested - 00:12:13
- ·PR #5512 (open): combines timeliness and inclusion-ness into a single field in the IL store - 00:13:06
- Client Updates:
- ·Teku FOCIL branch rebased on devnet-7; Lodestar is on devnet-8 trunk, making devnet-7 rebase impractical - 00:06:53
- ·Nethermind rebased on devnet-7; tests passing but may not reflect latest spec changes - 00:07:36
- Testing Progress:
- ·EL spec tests not yet updated to latest spec (Engine API changes); Jihoon to follow up at ACDE - 00:08:01
- ·FOCIL-only devnet planned; will base on latest stable devnet/testnet + latest FOCIL spec (Alpha 13, new release upcoming) - 00:23:43
Action Items
- •Jihoon - Update EL spec tests to reflect latest FOCIL spec changes (Engine API); raise at ACDE - 00:08:19
- •Jihoon - Open PR to key IL store by (slot, dependent_root) to align spec with implementation pruning patterns - 00:20:48
Key decisions
Cut a new FOCIL spec release beyond Alpha 13 to capture recent changes before FOCIL-only devnet
To capture recent changes before FOCIL-only devnet