Skip to content

Pullupstream/v0.3.0#21

Merged
arnaubennassar merged 26 commits intodevelopfrom
pullupstream/v0.0.3
Aug 31, 2023
Merged

Pullupstream/v0.3.0#21
arnaubennassar merged 26 commits intodevelopfrom
pullupstream/v0.0.3

Conversation

@arnaubennassar
Copy link
Collaborator

@arnaubennassar arnaubennassar commented Aug 29, 2023

Pull changes from zkevm-node @ release/v0.3.0

ARR552 and others added 25 commits August 14, 2023 16:35
* check flushid != 0 in sequencer

* Use f.halt instead of log.Fatal to report that flushid is 0

Co-authored-by: Toni Ramírez <58293609+ToniRamirezM@users.noreply.github.com>

* fix lint

---------

Co-authored-by: Toni Ramírez <58293609+ToniRamirezM@users.noreply.github.com>
0xPolygon#2412)

* fix http request instance null for websocket requests (0xPolygon#2385)

* fix ws subscribe to get filtered log notifications (0xPolygon#2396)
* add forced batches tx to addrQueue

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* refactor

* fix test

* fix test

* fix test

* fix test

* fix test

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

* fixe hash and from

* fixe hash and from

* fixe hash and from

* fixe hash and from

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* improve tests

* improve tests

* improve tests

* improve tests

* improve tests

* refactor

* refactor

* improve logs
Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>
…veGasPrice-nil-ref

bugifx: adding missing tx.BreakEvenGasPrice nil check
0xPolygon#2447)

* fix safe and finalized l2 block to consider l1 safe and finalized blocks respectively (0xPolygon#2245)

* fix and add tests for safe and finalized l2 blocks (0xPolygon#2424)
* wip

* new errors

* retry on executor db error

* new prover images

* fix comment

* update hasdh proto and prover images

* handle excutor db error

* update test

* update test

* update test

* update test

* refactor error check in unsigned tx
* reprocess full batch in parallel (sanity check)

* update doc

* update reprocessFullBatch logs
…ceipt.block_num (0xPolygon#2457)

* receipt deletion index

* receipt deletion index
* db table + tests

* GetForks func modified to get them by range

* Sync forkID

* forkIDIntervals and forkID genesis

* linter

* docs

* Avoid resetForkID in trustedNode

* fix test group 9

* suggestions

* doc and mocks
* Path snapshot command

* restore

* readme

* options used by dbeaver

* #2429_forkID_improvement: #2429_forkID_improvement:

* fix

* fix postgres version to v15

* fix permissionless init script
0xPolygon#2461)

* bugfix: removing measuring of metrics from async batch reprocessing for executor.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* fixing unit tests

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

---------

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>
@arnaubennassar arnaubennassar changed the title Pullupstream/v0.0.3 Pullupstream/v0.3.0 Aug 30, 2023
@IdrisHanafi
Copy link

Tested this version on internal devnet branch and successfully validated the following:

  • Bridging from L1 → L2
  • L2 → L2 Transactions
  • L2 → L1 Bridging
  • Validate Transaction Lifecycle (Batches gets trusted, virtualized, and verified)

Using Versions:

Service Version
CDK Validium Node This PR
CDK Data Availability Commit Hash a40e2f675523d0a5f0adcb8dc72ed036467f51bd
CDK Contracts Commit Hash d45deef41cb9520b7fb8b96aab99917cae98092b
zk Prover v2.2.0
zk Prover Config tags v2.0.0-RC4-fork.5
zkevm Bridge Service v0.2.0

Copy link

@IdrisHanafi IdrisHanafi left a comment

Choose a reason for hiding this comment

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

functionality wise....lgtm 🚀

@arnaubennassar arnaubennassar merged commit 47ae596 into develop Aug 31, 2023
@arnaubennassar arnaubennassar deleted the pullupstream/v0.0.3 branch August 31, 2023 11:54
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.

7 participants