Skip to content

Consensus-layer Call 140 #1129

Description

@ralexstokes

Consensus-layer Call 140

prev: call 139

Meeting Date/Time: Thursday 2024/8/22 at 14:00 UTC
Meeting Duration: 1.5 hours
stream

  1. https://github.com/ethereum/consensus-specs/releases/tag/v1.5.0-alpha.5
  2. Electra
  3. PeerDAS
    • implementation updates?
    • Blob uncoupling b/t EL and CL: EIP-7742 -- Pectra inclusion?
    • Blob configurability in clients
    • other open spec questions?
  4. Research, spec, etc.
    • F-star name
    • Probelab update on gossip analysis
  5. Open discussion/Closing remarks

Activity

  1. ralexstokes commented on Aug 13, 2024

    @ralexstokes
    MemberAuthor

    I'd like to revisit the name of the F-star for the hard fork after Pectra.

    Results integrated from various sources here: https://ethereum-magicians.org/t/f-star-name-for-consensus-layer-upgrade-after-electra/20285

    Would suggest we go with Fulu and use the portmanteau Fosaka

  2. lucassaldanha commented on Aug 13, 2024

    @lucassaldanha
    Contributor

    I would like to get some feedback on this PR based on @potuz's proposal of moving the execution layer request out of the execution payload.

    TLDR: we move deposit_requests, withdrawal_requests, and consolidation_requests into a new container ExecutionLayerRequests, and make them part of BeaconBlockBody, removing them from ExecutionPayload. This "restores" the assumption that the full Execution Payload isn't necessary for the CL performing state transition.

    This issue has more context and some discussion: ethereum/consensus-specs#3856

    And this is the PR with the proposed changes: ethereum/consensus-specs#3875

    p.s. unfortunately the call isn't at a suitable time for me so please direct any questions/comments to the issue. During the call, maybe @potuz or @mkalinin will be around and can help with more context and answering questions.

  3. mkalinin commented on Aug 15, 2024

    @mkalinin
    Contributor

    Want to give a quick overview of the following change:

  4. cortze commented on Aug 15, 2024

    @cortze

    I'd like to present a summary of the results that ProbeLab has done on the GossipSub domain. I will present our most relevant findings and the main takeaways from the study.
    Of course, all feedback and suggestions for further studies and improvement will be highly appreciated.

    Updates: I've uploaded the slides to IPFS, feel free to fetch them using the following CID: QmURBPigXY9LjGuumwRohrXfhhi26RZmWHKQT2RjcUEQAy

  5. ralexstokes commented on Aug 20, 2024

    @ralexstokes
    MemberAuthor

    blob flexibility in clients -- is the limit a compile-time constant? how easy is it to change?

    latest on EL/CL blob uncoupling

  6. lightclient commented on Aug 20, 2024

    @lightclient
    Member

    Hello can we have a final check on ethereum/execution-apis#565

  7. etan-status commented on Aug 22, 2024

    @etan-status

    Not in call myself, but would like to bring up:

    • Synchronously check all transactions to have non-zero length consensus-specs#3885 --
      • CLs may not do newPayload on every single block (to avoid engine API overhead), or may even support optimistic syncing without an EL (maintenance etc)
      • These CLs still need to do minimal checks on every block that doesn't go through newPayload, to ensure that the execution data in the beacon block is linked with the body.execution_payload.block_hash (so that it is guaranteed that everything building on this block_hash is also INVALID if there are asynchronous errors found later on by the EL). The minimal checks are:
        • No tx can be [] (the one added in the issue)
        • EL block hash must match the execution data
        • blob_kzg_commitments must match the blob_versioned_hashes from EIP-4844 blob transactions
      • The empty tx check is already present in most ELs, so no immediate rush. But still advising CLs to check their impls, even though this only affects non-validating clients (validating clients use newPayload and therefore have the EL perform the checks already).
    • Generally, the pitfalls with newPayload above are prompted by reluctance to move forward with issues such as SSZification. Aligning data formats across CL, EL, and engine API, while not being visible changes externally, are still important to keep complexity from growing.
    • EIP-7688 CL StableContainer kurtosis config updated for latest Lodestar, would be great to also add grandine, lighthouse, prysm eventually (config @ https://stabilitynow.box)
    • EIP-7732 enshrined proposer / builder separation: Personally in support of it (also benefits from SSZ 'optional's)
  8. akashkshirsagar31 commented on Aug 22, 2024

    @akashkshirsagar31
  9. ralexstokes commented on Aug 22, 2024

    @ralexstokes
    MemberAuthor

    closing in lieu of #1140

  10. added a commit that references this issue on Sep 4, 2024
  11. added
    ACDType: All Core Dev calls - execution & consensus
    ConsensusLayer: Issues that affect the consensus layer
    on Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    ACDType: All Core Dev calls - execution & consensusConsensusLayer: Issues that affect the consensus layer

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions