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

feat(L2): basic sequencer #586

Merged
merged 139 commits into from
Oct 8, 2024
Merged

feat(L2): basic sequencer #586

merged 139 commits into from
Oct 8, 2024

Conversation

ilitteri
Copy link
Contributor

@ilitteri ilitteri commented Sep 30, 2024

Warning

This PR depends on #575 and must be merged after it.

Motivation

Description

  • Adds block producer module
  • Implements Engine to advance the blockchain

fmoletta and others added 30 commits September 23, 2024 16:18
- `l1_tx_sender.rs` is the module that contains the logic for the L1TxSender, component of the operator in charge of sending transactions to L1, these are: commit and verify txs.
- `l1_watcher.rs` is the module that contains the logic for the L1Watcher, component of the operator that listens to specific L1 events and executes txs on L2 based on said events.
- The `prover` module is empty for the moment, but it'll contain the logic for getting proof data for block proof generation, and final proof submission.
@ManuelBilbao ManuelBilbao added the L2 label Oct 7, 2024
@ilitteri ilitteri marked this pull request as ready for review October 8, 2024 14:07
@ilitteri ilitteri requested a review from a team as a code owner October 8, 2024 14:07
@ilitteri ilitteri added this to the 1 - Welcome to the L2 milestone Oct 8, 2024
@ilitteri ilitteri linked an issue Oct 8, 2024 that may be closed by this pull request
@ilitteri ilitteri enabled auto-merge October 8, 2024 14:33
Copy link
Contributor

@fborello-lambda fborello-lambda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ilitteri ilitteri added this pull request to the merge queue Oct 8, 2024
Merged via the queue into main with commit 9145ff2 Oct 8, 2024
8 checks passed
@ilitteri ilitteri deleted the ethereum_rust_l2_consensus_mock branch October 8, 2024 15:04
mpaulucci pushed a commit to mpaulucci/lambda_ethereum_rust that referenced this pull request Oct 16, 2024
> [!WARNING]
> This PR depends on lambdaclass#575 and must be merged after it.

**Motivation**

<!-- Why does this pull request exist? What are its goals? -->

**Description**

<!-- A clear and concise general description of the changes this PR
introduces -->

- Adds block producer module
- Implements `Engine` to advance the blockchain

---------

Co-authored-by: fmoletta <fedemoletta@hotmail.com>
Co-authored-by: Manuel Bilbao <manuel.bilbao@lambdaclass.com>
Co-authored-by: Manuel Iñaki Bilbao <bilbaomanuel98@gmail.com>
mpaulucci pushed a commit to mpaulucci/lambda_ethereum_rust that referenced this pull request Oct 16, 2024
> [!WARNING]
> This PR depends on lambdaclass#575 and must be merged after it.

**Motivation**

<!-- Why does this pull request exist? What are its goals? -->

**Description**

<!-- A clear and concise general description of the changes this PR
introduces -->

- Adds block producer module
- Implements `Engine` to advance the blockchain

---------

Co-authored-by: fmoletta <fedemoletta@hotmail.com>
Co-authored-by: Manuel Bilbao <manuel.bilbao@lambdaclass.com>
Co-authored-by: Manuel Iñaki Bilbao <bilbaomanuel98@gmail.com>
mpaulucci pushed a commit that referenced this pull request Oct 16, 2024
> [!WARNING]
> This PR depends on #575 and must be merged after it.

**Motivation**

<!-- Why does this pull request exist? What are its goals? -->

**Description**

<!-- A clear and concise general description of the changes this PR
introduces -->

- Adds block producer module
- Implements `Engine` to advance the blockchain

---------

Co-authored-by: fmoletta <fedemoletta@hotmail.com>
Co-authored-by: Manuel Bilbao <manuel.bilbao@lambdaclass.com>
Co-authored-by: Manuel Iñaki Bilbao <bilbaomanuel98@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

L2: Implement basic block producer
4 participants