forked from ethereum-optimism/optimism
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Mytest #1
Open
curryxbo
wants to merge
412
commits into
mybase
Choose a base branch
from
mytest
base: mybase
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
…d-correct-length fix(fd): use correct fn for getTotalElements
More accurately computes the gas limit for a migrated withdrawal. This is useful to prevent the gaslimit for large transactions from going over the consensus level block gas limit. It is unlikely for a transaction to go over the block gas limit, but this is a preventative fix just in case.
…migration-address-cheat op-wheel: update OVM owners cheat-code for migration testing
…arallel_op_e2e op-e2e: Run CI in parallel
…is-correct-token-pair contracts-bedrock: fix standard bridge token pair checking
…remove-ownable-checks-l1xdm contracts-bedrock: remove deployment ownable check l1xdm
…c/module-fix fix(atst): Fix module and main in package.json
…ogger op-node/service: Deduplicate logger + Better defaults
…gaslimit migration: update migrated withdrawal gas limit
Removes an old package. Now included in chain-mon instead.
…ocs-stack-algolia-key fix(docs): update algolia search key
…step-rm maint: remove old two-step-monitor package
…ndabot/npm_and_yarn/docs/op-stack/valine-1.5.1 build(deps): bump valine from 1.4.16 to 1.5.1 in /docs/op-stack
…ndabot/npm_and_yarn/docs/op-stack/minimatch-3.1.2 build(deps): bump minimatch from 3.0.4 to 3.1.2 in /docs/op-stack
Bumps [terser](https://github.com/terser/terser) from 4.8.0 to 4.8.1. - [Release notes](https://github.com/terser/terser/releases) - [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md) - [Commits](terser/terser@v4.8.0...v4.8.1) --- updated-dependencies: - dependency-name: terser dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
…by/ctb/speedy-diff-testing feat(ctb): Optimize differential testing
…ndabot/npm_and_yarn/docs/op-stack/terser-4.8.1 build(deps): bump terser from 4.8.0 to 4.8.1 in /docs/op-stack
…migrate-goerli-proxy-artifacts contracts-bedrock: migrate goerli proxy deploy artifacts
…ella-fix op-node: support shanghai L1 blockhash input verification
…ell/batcher/config feat(op-batcher): Batcher Config Validation
…by/op-node/alt-rpc-sync feat(op-node): Add alternative backup sync method via RPC
After some research, I discovered that we can iterate over Geth's storage in parallel as long as we don't share the state database. This PR updates the OVM ETH check script to parallelize state iteration across 64 workers. The parallelization works by partitioning storage keyspace based on the number of workers. To optimize further, I also moved the state balance checking into the main iteration loop to avoid unnecessary iteration. Note that state mutation must be done serially. Overall, this should safe about 40 minutes during the migration. This PR was tested by running it against a mainnet data directory. The entire process took approximately one hour. As part of this testing I discovered an invalid check in `check.go`, which was verifying the wrong storage slot in the withdrawals check function. This has been fixed.I confirmed with Mark that the updated `MessageSender` value is the correct one to be checking for. The filtering code was correct from the beginning.
…/parallel-migration op-chain-ops: Add parallel migration
Trigger Releases
…atch_decoder_force_close batch_decoder: Force close channels
…geset-release/master Version Packages
Develop -> Master
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.