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

fix: cdk node l1infotree sync configuration #380

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

leovct
Copy link
Member

@leovct leovct commented Nov 15, 2024

Description

While deploying the stack on external L1s like Sepolia using kurtosis-cdk, some users noticed that the cdk node was syncing from genesis instead of syncing from the rollup manager deployment block. It turns out the cdk node configuration is a bit outdated. Some settings are not set, instead, we rely on default values. For example, the L1 InfoTreeSync initial block is set to zero by default which explains why the cdk node syncs from the genesis block. This PR addresses this specific issue.

  • Minor: Simplify the way contract versions is set in the config.
  • Next steps: update and document the cdk node configuration.

References (if applicable)

https://discord.com/channels/893151281848406016/1306202853244665867/1306593102902857780

@leovct leovct requested a review from a team November 15, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant