Skip to content
This repository has been archived by the owner on Dec 5, 2021. It is now read-only.

Add regenesis #237

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from
Open

Add regenesis #237

wants to merge 27 commits into from

Conversation

boyuan-chen
Copy link

@boyuan-chen boyuan-chen commented Jul 27, 2021

TODO:

  • Add the new OVM_L1CrossDomainMessenger
  • Add the script to speed up the pending transactions

@boyuan-chen boyuan-chen changed the title Add regenesis Add regenesis[DON'T MERGE] Jul 30, 2021
@github-actions github-actions bot added the M-ops label Jul 30, 2021
@boyuan-chen boyuan-chen changed the title Add regenesis[DON'T MERGE] Add regenesis Aug 2, 2021
Copy link

@souradeep-das souradeep-das left a comment

Choose a reason for hiding this comment

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

nice! looks good to me

packages/omgx/regenesis/src/service.ts Outdated Show resolved Hide resolved
@@ -0,0 +1,36 @@
export const makeRegenesisDump = (dump:any, genesis: any) => {
Copy link

Choose a reason for hiding this comment

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

This is the script that we have used to make the regenesis: https://github.com/ethereum-optimism/scripts/blob/main/scripts/state-surgery.js. It currently includes some specific logic for the previous regenesis, so look at its history if you are interested in seeing a more pure form.

Whenever there is another regenesis, we will move it into the monorepo and be very thorough around using it and documenting it.

@github-actions github-actions bot removed the M-ops label Sep 12, 2021
@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2021

Codecov Report

Merging #237 (198cb28) into develop (d940b26) will decrease coverage by 3.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #237      +/-   ##
===========================================
- Coverage    79.16%   76.08%   -3.08%     
===========================================
  Files           68       81      +13     
  Lines         2721     3036     +315     
  Branches       421      469      +48     
===========================================
+ Hits          2154     2310     +156     
- Misses         567      726     +159     
Flag Coverage Δ
batch-submitter 59.44% <ø> (ø)
contracts 86.05% <ø> (ø)
core-utils 68.16% <ø> (ø)
data-transport-layer 37.68% <ø> (?)
message-relayer 72.22% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../l1-ingestion/handlers/sequencer-batch-appended.ts 20.58% <0.00%> (ø)
packages/message-relayer/hardhat.config.ts 100.00% <0.00%> (ø)
packages/message-relayer/src/relay-tx.ts 71.15% <0.00%> (ø)
...kages/data-transport-layer/src/utils/validation.ts 10.71% <0.00%> (ø)
packages/data-transport-layer/src/utils/index.ts 100.00% <0.00%> (ø)
...ckages/data-transport-layer/src/utils/constants.ts 100.00% <0.00%> (ø)
...ckages/data-transport-layer/src/utils/contracts.ts 22.72% <0.00%> (ø)
.../src/services/l2-ingestion/handlers/transaction.ts 60.86% <0.00%> (ø)
...layer/src/services/l1-ingestion/handlers/errors.ts 33.33% <0.00%> (ø)
...ices/l1-ingestion/handlers/state-batch-appended.ts 68.18% <0.00%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d940b26...198cb28. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants