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.

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

AggLayer integration #71

Closed
wants to merge 64 commits into from
Closed

AggLayer integration #71

wants to merge 64 commits into from

Conversation

vcastellm
Copy link

This PR adds config options and logic to send proofs to AggLayer

agnusmor and others added 30 commits January 23, 2024 18:35
* fix Leaves smtproof

* pointer protection

* fix unit test
…0xPolygonHermez#3129)

* add l2block log dump when storing error. refactor ProcessBatchV2 logs

* add AddL2Block log

* fix index AddL2Block log

* use aux variable to avoid reusing poiner

---------

Co-authored-by: Toni Ramírez <[email protected]>
…storing L2Block. Logs changes (0xPolygonHermez#3134)

* fix l2 hash when to is nill

* get l2 hash from executor

* fix use tx L2 hash returned by the executor when storing L2Block. Logs changes

* fix linter

* add L2block tracking number to process tx logs

* fix ProcessBatchV2 log format

* fix UTs. set useMainExecGenerated to true

* set prover image to v4.0.0-RC30

* test setting useMainExecGenerated = false

* fix independent_test

* fix non-e2e tests

---------

Co-authored-by: Toni Ramírez <[email protected]>
* if got an error getting lastBlock produce a SIGSEGV

* reduce info logs

* fix unittest
* fix missingBookMark

* fix missingBookMark

* fixes

* fix

* fix

* fix seq-sender config

* fixes

* doc
* improve open new wipL2Block logs

* fix wipL2Block deltaTimestamp
…in the pool (0xPolygonHermez#3156)

* set DEFAULT 0 when creatingtransaction.used_sha256_hashes field in the pool

* update prover image to v4.0.0
* add cardona testnet (0xPolygonHermez#2909)
* update config
* update cardona genesis

---------

Co-authored-by: Thiago Coimbra Lemos <[email protected]>
* new fields in stream

* update test

* remove unused constant

* add UpgradeEtrogBatchNumber handling to stream

* doc

* add log to test

* add log to test

* add protection

* add check

* fix

* fix
* fix case emtpy batch and unittest

* fix error in conversion

* fix unittest
Copy link
Collaborator

@arnaubennassar arnaubennassar left a comment

Choose a reason for hiding this comment

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

fix the linter, and minor suiggestions

cmd/run.go Outdated Show resolved Hide resolved
cmd/run.go Outdated Show resolved Hide resolved
config/config.go Show resolved Hide resolved
Copy link
Collaborator

@goran-ethernal goran-ethernal left a comment

Choose a reason for hiding this comment

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

Looks good to me. Linter errors are left, and I think, node-config-schema.json needs to be changed because of the added SettlementBackend.

Base automatically changed from pullupstream/v0.5.6 to develop February 8, 2024 15:01
This PR adds config options and logic to send proofs to AggLayer
@vcastellm vcastellm force-pushed the agglayer-integration branch from 2798c5b to 6fbef6a Compare February 8, 2024 15:24
@vcastellm vcastellm closed this Feb 8, 2024
@vcastellm vcastellm deleted the agglayer-integration branch February 8, 2024 15:28
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.

9 participants