Skip to content

Conversation

@cdiielsi
Copy link
Contributor

@cdiielsi cdiielsi commented Nov 6, 2025

Motivation

PR #5004 led to changes in init_l2() that weren't properly updated, so now when initializing L2 there are 2 Peer Tables.

Description

This pr uses the Peer Table created in the first place to initialize the Peer Handler instead of creating a new one.
It also updates the gen server initialization for the gen server dummy.

@github-actions github-actions bot added the L2 Rollup client label Nov 6, 2025
@cdiielsi cdiielsi marked this pull request as ready for review November 7, 2025 13:23
@cdiielsi cdiielsi requested a review from a team as a code owner November 7, 2025 13:23
@cdiielsi cdiielsi added this pull request to the merge queue Nov 7, 2025
Merged via the queue into main with commit a891b78 Nov 7, 2025
49 checks passed
@cdiielsi cdiielsi deleted the fix_init_l2_duplicated_peer_table branch November 7, 2025 14:31
@github-project-automation github-project-automation bot moved this to Done in ethrex_l2 Nov 7, 2025
xqft pushed a commit that referenced this pull request Nov 11, 2025
**Motivation**

PR #5004 led to changes in `init_l2()` that weren't properly updated, so
now when initializing L2 there are 2 Peer Tables.

**Description**

This pr uses the Peer Table created in the first place to initialize the
Peer Handler instead of creating a new one.
It also updates the gen server initialization for the gen server dummy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L2 Rollup client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants