Skip to content

Conversation

zimpha
Copy link
Member

@zimpha zimpha commented Feb 8, 2023

This PR implements the contract parts of new bridge design. The details can be found here.

This PR is a breaking change.

@zimpha zimpha changed the title Feat/new bridge contracts feat(contracts): new bridge contracts Feb 8, 2023
@icemelon icemelon force-pushed the feat/new_bridge_contracts branch from d749daa to bc207b0 Compare February 8, 2023 19:33
@ChuhanJin
Copy link
Contributor

46% (+0.19%) vs master 46%

@ChuhanJin
Copy link
Contributor

47% (+0.16%) vs master 46%

@ChuhanJin
Copy link
Contributor

47% (0.0%) vs master 47%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (+0.04%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (+0.04%) vs master 46%

@icemelon icemelon marked this pull request as ready for review February 15, 2023 01:12
@ChuhanJin
Copy link
Contributor

46% (-0.03%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@zimpha zimpha force-pushed the feat/new_bridge_contracts branch from 19d630e to 6491392 Compare February 19, 2023 17:26
@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

icemelon
icemelon previously approved these changes Feb 20, 2023
Copy link
Member

@icemelon icemelon left a comment

Choose a reason for hiding this comment

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

I didn't take a very close look the deployment scripts as I'm not very familiar with that part. @Thegaram have you reviewed the deployment part?
Otherwise, this PR looks good to me now.

@ChuhanJin
Copy link
Contributor

46% (0.0%) vs master 46%

@icemelon icemelon merged commit 7e99c51 into alpha Feb 20, 2023
@icemelon icemelon deleted the feat/new_bridge_contracts branch February 20, 2023 21:44
ChuhanJin pushed a commit that referenced this pull request Mar 15, 2023
# This is the 1st commit message:

feat: add monitor metrics (#262)

Co-authored-by: colinlyguo <[email protected]>
Co-authored-by: maskpp <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: Péter Garamvölgyi <[email protected]>
# This is the commit message #2:

refactor(bridge): remove layer1 client in in layer1 relayer constructor (#274)

Co-authored-by: vincent <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
# This is the commit message #3:

fix: add gas multiplier (#275)


# This is the commit message #4:

feat(libzkp): use dylib instead of staticlib  (#266)

Co-authored-by: maskpp <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: colin <[email protected]>
Co-authored-by: colinlyguo <[email protected]>
# This is the commit message #5:

Revert "fix: add gas multiplier (#275)" (#279)


# This is the commit message #6:

build: add nightly-2022-12-10 rust-related builder image (#282)


# This is the commit message #7:

fix(bridge): compatible with DynamicFeeTxType not supported chain (#280)

Co-authored-by: colinlyguo <[email protected]>
# This is the commit message #8:

feat(contract): enable whitelist relayer (#272)

Co-authored-by: Péter Garamvölgyi <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
# This is the commit message #9:

perf(bridge): execute relayer loops independently (#258)

Co-authored-by: colin <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
# This is the commit message #10:

feat(bridge): confirm block based on "safe" and "finalized" tags (#265)


# This is the commit message #11:

feat: allow to override L2 deployment when address is provided (#293)


# This is the commit message #12:

feat(contracts): Add fee vault (#223)


# This is the commit message #13:

feat(confirmations): Upgrade confirm (#291)


# This is the commit message #14:

feat(coordinator): Enable set ws compression level. (#292)


# This is the commit message #15:

feat(roller&coordinator): upgrade lizkp to zkevm-0215 version (#281)

Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: xinran chen <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
# This is the commit message #16:

build: update version to `alpha-v1.0` (#301)


# This is the commit message #17:

feat: import genesis batch during startup (#299)

Co-authored-by: HAOYUatHZ <[email protected]>
# This is the commit message #18:

feat(contracts): new bridge contracts (#288)

Co-authored-by: Haichen Shen <[email protected]>
Co-authored-by: Thegaram <[email protected]>
Co-authored-by: colin <[email protected]>
# This is the commit message #19:

chore: upgrade l2geth dependency for trace type (#304)
ChuhanJin pushed a commit that referenced this pull request Mar 15, 2023
Co-authored-by: colinlyguo <[email protected]>
Co-authored-by: maskpp <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: Péter Garamvölgyi <[email protected]>

refactor(bridge): remove layer1 client in in layer1 relayer constructor (#274)

Co-authored-by: vincent <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>

fix: add gas multiplier (#275)

feat(libzkp): use dylib instead of staticlib  (#266)

Co-authored-by: maskpp <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: colin <[email protected]>
Co-authored-by: colinlyguo <[email protected]>

Revert "fix: add gas multiplier (#275)" (#279)

build: add nightly-2022-12-10 rust-related builder image (#282)

fix(bridge): compatible with DynamicFeeTxType not supported chain (#280)

Co-authored-by: colinlyguo <[email protected]>

feat(contract): enable whitelist relayer (#272)

Co-authored-by: Péter Garamvölgyi <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>

perf(bridge): execute relayer loops independently (#258)

Co-authored-by: colin <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>

feat(bridge): confirm block based on "safe" and "finalized" tags (#265)

feat: allow to override L2 deployment when address is provided (#293)

feat(contracts): Add fee vault (#223)

feat(confirmations): Upgrade confirm (#291)

feat(coordinator): Enable set ws compression level. (#292)

feat(roller&coordinator): upgrade lizkp to zkevm-0215 version (#281)

Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: xinran chen <[email protected]>
Co-authored-by: Ubuntu <[email protected]>

build: update version to `alpha-v1.0` (#301)

feat: import genesis batch during startup (#299)

Co-authored-by: HAOYUatHZ <[email protected]>

feat(contracts): new bridge contracts (#288)

Co-authored-by: Haichen Shen <[email protected]>
Co-authored-by: Thegaram <[email protected]>
Co-authored-by: colin <[email protected]>

chore: upgrade l2geth dependency for trace type (#304)

feat(bridge): update the watcher and relayer based on the new contract (#305)

Co-authored-by: colinlyguo <[email protected]>
Co-authored-by: zimpha <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>

fix incorrect block order during batch recovery (#311)

fix(bridge): fix batch proposer (#312)

Co-authored-by: HAOYUatHZ <[email protected]>
Co-authored-by: Péter Garamvölgyi <[email protected]>

fix: Flush buffered writer (#314)

fix(config): fix typos (#315)

feat(bridge): only update gas price oracle for exceeding diff threshold (#319)

fix(bridge): fix typos (#321)

feat(batch proposer): add time limit to commit batches (#323)

fix(db): fix `SetMaxOpenConns` (#328)

Fix(zkevm): fix zkevm bug for goerli.  (#334)

fix(build): jenkinsfile tag job optimized and fix (#331)

Co-authored-by: vincent <[email protected]>
Co-authored-by: HAOYUatHZ <[email protected]>

fix: reduce finalize batch tx frequency (#332)

alpha inegration with multibin

fix makefile

fix lint

fix lint

try fix commit

op

fix lint

goimports

trigger ci

bugfix(libzkp): fix difficulty (#343)

fix(contract): forbid to call message queue and l2 messenger from l1 (#341)

Co-authored-by: HAOYUatHZ <[email protected]>

fix(roller): fix stack bug (#320)

Co-authored-by: HAOYUatHZ <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants