Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

ops: Fix devnet #321

Merged
merged 3 commits into from
Apr 6, 2022
Merged

ops: Fix devnet #321

merged 3 commits into from
Apr 6, 2022

Conversation

mslipper
Copy link
Collaborator

@mslipper mslipper commented Apr 6, 2022

  • Fixes the devnet config to correctly generate rollup/genesis files.
  • Fixes a bug that causes the rollup node to get stuck when there's a large timestamp delta between genesis and the current time, as is the case in the devnet.

@mslipper
ops: Fix devnet
38a3930 
- Fixes the devnet config to correctly generate rollup/genesis files.
- Fixes a bug that causes the rollup node to get stuck when there's a large timestamp delta between genesis and the current time, as is the case in the devnet.
@mslipper @protolambda
Update opnode/rollup/driver/state.go
00c593b 
Co-authored-by: Diederik Loerakker <proto@protolambda.com>
protolambda
protolambda reviewed Apr 6, 2022
View reviewed changes
ops/rollup.json Outdated Show resolved Hide resolved
trianglesphere
trianglesphere approved these changes Apr 6, 2022
View reviewed changes
Copy link
Contributor

@trianglesphere trianglesphere left a comment

Choose a reason for hiding this comment

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

I think that the new definition of L1 rollup start makes sense and we should change to it. It does have some knock on implications in how we handle the L2 start and specs definitions that we need to fix

@protolambda protolambda mentioned this pull request Apr 6, 2022
Open
@protolambda protolambda merged commit 83fc45d into ethereum-optimism:main Apr 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tynes tynes tynes left review comments

@trianglesphere trianglesphere trianglesphere approved these changes

@protolambda protolambda protolambda approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mslipper @tynes @trianglesphere @protolambda