forked from 0xPolygonHermez/zkevm-node
-
Notifications
You must be signed in to change notification settings - Fork 121
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
Pullupstream/v0.5.0 rc23 #61
Merged
arnaubennassar
merged 92 commits into
pullupstream/v0.5.0-RC4
from
pullupstream/v0.5.0-RC23
Feb 1, 2024
Merged
Pullupstream/v0.5.0 rc23 #61
arnaubennassar
merged 92 commits into
pullupstream/v0.5.0-RC4
from
pullupstream/v0.5.0-RC23
Feb 1, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix l1InfoTreeData when syncing * fix * remove leaves in forcedBatches
…olygonHermez#3035) * add zkevm_EstimateFee endpoint (0xPolygonHermez#3000) * update pool mock
* fix e2e group 6 * If a L1 virtual batch detects missmatch with state batch doesn't create a entry in trusted_reorg table if the batch is WIP * Add new Makefile target to show dockers logs
…3043) * new fields in stream * read blockHashL1 * fix sql query * fix sql query * logs * logs * logs * logs * logs * logs * logs * fix test * fix linter * update doc * fill l1InfoRoot
* fix inforoot * fixes --------- Co-authored-by: Toni Ramírez <[email protected]>
* fix l1InfoRoot in sync, aggregator and state * fix
* add more unittest for trusted sync * new uniitest for trusted sync and control a case of not able to close a batch because the data doesnt match * fix batch that dont need process but could closed a batch * fix status update * fix desync problem between producer and consumer
…lygonHermez#3112) * fix smtProof * remove logs * test force l1InfoRootIndex to 0 on startup * fix close batch due to exhausted resource * remove force l1InfoRootIndex to 0 on startup * update prover image to v4.0.0-RC28 * update prover image to v4.0.0-RC29
* fix new wipL2block on finalizeBatch function * fix sustract wipL2Block usedResources when opening new WIPbatch * fix openNewWipL2Block * fix openNewWipL2Block * update prover image to v4.0.0-RC27 * Update prover image to v4.0.0-RC28 * update prover image to v4.0.0-RC27
…ygonHermez#2938) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.60.0 to 1.60.1. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.60.0...v1.60.1) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* update prover image * update test vectors
…z#3105) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.17.0 to 1.18.0. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.17.0...v1.18.0) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…rmez#3106) Bumps [golang.org/x/net](https://github.com/golang/net) from 0.19.0 to 0.20.0. - [Commits](golang/net@v0.19.0...v0.20.0) --- updated-dependencies: - dependency-name: golang.org/x/net dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* wip * Fix aggregator * change name to UpgradeEtrogBatchNumber * doc
… number to L2 block. Several refactors (0xPolygonHermez#3118) * add L2 block sequential process. Execute empty L2 block. Add tracking number to L2 block. Several refactors * fix finalizer UT
christophercampbell
approved these changes
Jan 30, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Stefan-Ethernal
approved these changes
Jan 31, 2024
rachit77
approved these changes
Jan 31, 2024
goran-ethernal
approved these changes
Jan 31, 2024
@Stefan-Ethernal while I consider your feedback to be on point, this is upstream code, and I prefer not to modify it |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.