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.

Sign up for GitHub

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

Jump to bottom

synchronizer: support to elderberry fork #3300

Closed
wants to merge 46 commits into from
Closed

synchronizer: support to elderberry fork #3300

wants to merge 46 commits into from

Conversation

joanestebanr
Copy link
Contributor

Closes #3299

What does this PR do?

Reviewers

Main reviewers:

Codeowner reviewers:

  • @-Alice
  • @-Bob

joanestebanr and others added 30 commits January 29, 2024 10:17
@joanestebanr
EncodeBatchV2 allow to set Tx in binary (#3143)
fd05d50 
* EncodeBatchV2 allow to set Tx in binary
* actions/setup-go to v5 to avoid use node 16
@joanestebanr
synchronizer: unittest and remove pool_reorg (#3111)
b938572 
* removed reorgPool call
* add test for l1 sequenced batches
* removed panic after halt call as PR comments ask for
@tclemos
avoid unnecessary get code call when address doesn't have code hash (#…
3f537af 
…3170)
@agnusmor
fix parentHash when storing a new L2 block (#3171) (#3174)
188969b
@ARR552
new geth version v1.13.11 (#3179)
aed5088 
* new geth version v1.13.11
@ARR552
Fix/rom error (#3177) (#3178)
2f51cda 
* fix rom error

* linter

* fix
@joanestebanr
synchronizer: update fromTrusted fix cache update (#3169) (#3183)
cb2424a 
* fix cache update and sync from Trusted of closed batch
@tclemos
fix l2 block gas price to use value returned by the executor (#3186)
8c659f2
@IdrisHanafi
feat: optimize purge txn pool query (#3137)
3212dc4 
* feat: optimize purge txn pool query

* fix: add new method to state interface

* feat: generate new state mocks
@agnusmor
Add sanity check: last L2 block timestamp in the sequence is also L1B…
9f6ba6a 
…lockTimestampMargin seconds behind "now" (#3181) (#3189)

* add sanity check L1BlockTimestampMargin vs now

* Update prover image to v4.0.4
@agnusmor
Fix isSynced function (#3188) (#3194)
2e29d41 
* fix isSynced

* fix linter

* fix logs

* halt on sanity check error
@tclemos
fix migration down; check for greater forkIDs instead of the next +1 (#…
fd78640 
…3182)
@joanestebanr
cherry-pick Fix/3200 l1inforoot error on reorg (#3201) (#3207)
31959f5 
* Fix/3200 l1inforoot error on reorg (#3201)
* L1InfooTree cache is calculated when needed and is discard when a reorg happens
@joanestebanr
Feature/3173 sync trusted state in networks previous to ETROG (#3175) (…
daf4094 
…#3214)
@joanestebanr
Fix/3203 panic if batch process fail and must be closed (#3206) (#3216)
d6d4b96 
* synchronizer avoid panic if process trusted batch fails
@joanestebanr
Fix/3164 eth syncing dont return expected values (#3165) (#3193)
1dfa467
@ToniRamirezM
Fix etrog im state root (#3222)
cae61bd 
* fix stream

* fix stream

* fix stream

* fix stream

* linter
@agnusmor
Fix L2 block gas limit to 2**50 (1125899906842624) (#3219) (#3223)
68338dc 
* set l2 block gas limit to 2**50 (1125899906842624)

* limit L2 block gasLimit in StoreL2Block function
@joanestebanr
permissionless: removed compatibility with trusted Incaberry (#3218)
2ee5030 
* removed class Processor Selector, and set etrog as processor
@joanestebanr
change 2^64-1 for an esmitation of blocks (#3220) (#3226)
7756dd3
@joanestebanr
synchronizer mode sequential by default (#3227)
25d6c61
@agnusmor
add retries in is Synced function when last virtual batch is greater …
54ef1c4 
…than last SC sequenced batcj (#3225) (#3231)
@tclemos
Cherry pick PRs #3209 and #3197 into develop (#3213)
44248e0
@agnusmor
sequence batch only when sanity check is done. Do fatal for some halt…
3977793 
… events. Several fixes (#3240)
@ToniRamirezM
fix im state root in stream (#3192) (#3202)
e2c4b73
@ToniRamirezM
get GER for l2block on Etrog (#3238) (#3242)
129cece
@joanestebanr
Fix/3233 the etrog event UpdateEtrogSequence doesnt store the GER in …
445bf0b 
…batch table (#3234) (#3244)
@dependabot
build(deps): bump github.com/prometheus/common from 0.45.0 to 0.46.0 (#…
f7e4167 
…3119)

Bumps [github.com/prometheus/common](/~https://github.com/prometheus/common) from 0.45.0 to 0.46.0.
- [Release notes](/~https://github.com/prometheus/common/releases)
- [Commits](prometheus/common@v0.45.0...v0.46.0)

---
updated-dependencies:
- dependency-name: github.com/prometheus/common
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump golang.org/x/sync from 0.5.0 to 0.6.0 (#3121)
40f6f27 
Bumps [golang.org/x/sync](/~https://github.com/golang/sync) from 0.5.0 to 0.6.0.
- [Commits](golang/sync@v0.5.0...v0.6.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump github.com/urfave/cli/v2 from 2.26.0 to 2.27.1 (#3120)
233ec7c 
Bumps [github.com/urfave/cli/v2](/~https://github.com/urfave/cli) from 2.26.0 to 2.27.1.
- [Release notes](/~https://github.com/urfave/cli/releases)
- [Changelog](/~https://github.com/urfave/cli/blob/main/docs/CHANGELOG.md)
- [Commits](urfave/cli@v2.26.0...v2.27.1)

---
updated-dependencies:
- dependency-name: github.com/urfave/cli/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
tclemos and others added 13 commits February 9, 2024 16:17
@tclemos
add zkevm_estimateGasPrice (#3248)
2e79d28
@tclemos
refactor gasUsed to validate break even gas price to match sequencer …
361cb86 
…logic (#3239)
@dependabot
build(deps): bump golang.org/x/crypto from 0.18.0 to 0.19.0 (#3253)
025e5d9 
Bumps [golang.org/x/crypto](/~https://github.com/golang/crypto) from 0.18.0 to 0.19.0.
- [Commits](golang/crypto@v0.18.0...v0.19.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump github.com/0xPolygonHermez/zkevm-data-streamer (#3251)
cca6ef0 
Bumps [github.com/0xPolygonHermez/zkevm-data-streamer](/~https://github.com/0xPolygonHermez/zkevm-data-streamer) from 0.1.18 to 0.1.19.
- [Release notes](/~https://github.com/0xPolygonHermez/zkevm-data-streamer/releases)
- [Commits](0xPolygon/zkevm-data-streamer@v0.1.18...v0.1.19)

---
updated-dependencies:
- dependency-name: github.com/0xPolygonHermez/zkevm-data-streamer
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump github.com/google/uuid from 1.5.0 to 1.6.0 (#3250)
4b6dee6 
Bumps [github.com/google/uuid](/~https://github.com/google/uuid) from 1.5.0 to 1.6.0.
- [Release notes](/~https://github.com/google/uuid/releases)
- [Changelog](/~https://github.com/google/uuid/blob/master/CHANGELOG.md)
- [Commits](google/uuid@v1.5.0...v1.6.0)

---
updated-dependencies:
- dependency-name: github.com/google/uuid
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump golang.org/x/net from 0.20.0 to 0.21.0 (#3252)
f12cdc3 
Bumps [golang.org/x/net](/~https://github.com/golang/net) from 0.20.0 to 0.21.0.
- [Commits](golang/net@v0.20.0...v0.21.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@invocamanman
fix loginfof (#3261)
b22cf41
@dependabot
build(deps): bump google.golang.org/grpc from 1.61.0 to 1.61.1 (#3268)
4e298ae 
Bumps [google.golang.org/grpc](/~https://github.com/grpc/grpc-go) from 1.61.0 to 1.61.1.
- [Release notes](/~https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.61.0...v1.61.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@agnusmor
remove state.batch section from environments config files (#3230) (#3247
7719a86 
)
@agnusmor @ToniRamirezM
Cherry-pick #3263, #3266: Fix node OOC. Fix error handling. Sanity ch…
1ba1e80 
…eck closing empty WIP batch (#3279)

* fix node OOC (#3263)

* fix error handling (#3266)

* added sanity check closing an empty batch

---------

Co-authored-by: Toni Ramírez <58293609+ToniRamirezM@users.noreply.github.com>
@dependabot
build(deps): bump github.com/prometheus/common from 0.46.0 to 0.47.0 (#…
f92dca9 
…3282)

Bumps [github.com/prometheus/common](/~https://github.com/prometheus/common) from 0.46.0 to 0.47.0.
- [Release notes](/~https://github.com/prometheus/common/releases)
- [Commits](prometheus/common@v0.46.0...v0.47.0)

---
updated-dependencies:
- dependency-name: github.com/prometheus/common
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@ToniRamirezM
fix stream generation for old fork Ids (#3284)
bddde4c
@joanestebanr
add elderberry base, pending SMC
f9ac2ae
@joanestebanr joanestebanr added this to the v0.6.0 milestone Feb 19, 2024
@joanestebanr joanestebanr self-assigned this Feb 19, 2024
@joanestebanr joanestebanr requested a review from ARR552 as a code owner February 19, 2024 12:13
@cla-bot cla-bot bot added the cla-signed label Feb 19, 2024
@joanestebanr joanestebanr changed the base branch from develop to release/v0.5.12 February 19, 2024 12:14
@joanestebanr joanestebanr changed the base branch from release/v0.5.12 to release/v0.6.0 February 19, 2024 12:15
@joanestebanr joanestebanr marked this pull request as draft February 19, 2024 12:15
@joanestebanr
Copy link
Contributor Author

The branch was created from develop instead of v0.6.0.
Create a new PR #3301

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ARR552 ARR552 Awaiting requested review from ARR552 ARR552 is a code owner

@tclemos tclemos Awaiting requested review from tclemos tclemos is a code owner

@ToniRamirezM ToniRamirezM Awaiting requested review from ToniRamirezM ToniRamirezM is a code owner

@dpunish3r dpunish3r Awaiting requested review from dpunish3r

@agnusmor agnusmor Awaiting requested review from agnusmor agnusmor is a code owner

Assignees

@joanestebanr joanestebanr

Labels
cla-signed etherman synchronizer zkevm-bridge-sync-rpc
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging this pull request may close these issues.

elderberry: synchronizer: new event for SequencedBatch
7 participants
@joanestebanr @tclemos @agnusmor @ARR552 @IdrisHanafi @ToniRamirezM @invocamanman