Merge pull request #7 from bnb-chain/merge_v0.2.0-beta.7

Merge v0.2.0 beta.7
bnb-chain · May 11, 2024 · aadf535 · aadf535
2 parents ac29b4b + a44e085
commit aadf535
Show file tree

Hide file tree

Showing 564 changed files with 20,879 additions and 14,285 deletions.
diff --git a/.codespellrc b/.codespellrc
@@ -0,0 +1,3 @@
+[codespell]
+skip = .git,target,./crates/storage/libmdbx-rs/mdbx-sys/libmdbx,Cargo.toml,Cargo.lock
+ignore-words-list = crate,ser,ratatui
diff --git a/.github/ISSUE_TEMPLATE/bug.yml b/.github/ISSUE_TEMPLATE/bug.yml
@@ -6,15 +6,15 @@ body:
     attributes:
       value: |
         Thanks for taking the time to fill out this bug report! Please provide as much detail as possible.
-        
+
         If you believe you have found a vulnerability, please provide details [here](mailto:[email protected]) instead.
   - type: textarea
     id: what-happened
     attributes:
       label: Describe the bug
       description: |
         A clear and concise description of what the bug is.
-        
+
         If the bug is in a crate you are using (i.e. you are not running the standard `reth` binary) please mention that as well.
     validations:
       required: true
@@ -25,7 +25,7 @@ body:
       description: Please provide any steps you think might be relevant to reproduce the bug.
       placeholder: |
         Steps to reproduce:
-        
+
         1. Start '...'
         2. Then '...'
         3. Check '...'
@@ -76,6 +76,13 @@ body:
       description: This can be obtained with `reth db version`
     validations:
       required: true
+  - type: textarea
+    id: network
+    attributes:
+      label: Which chain / network are you on?
+      description: This is the argument you pass to `reth --chain`. If you are using `--dev`, type in 'dev' here. If you are not running with `--chain` or `--dev` then it is mainnet.
+    validations:
+      required: true
   - type: dropdown
     id: node-type
     attributes:

diff --git a/.github/scripts/label_pr.js b/.github/scripts/label_pr.js
@@ -8,18 +8,28 @@ function shouldIncludeLabel (label) {
     return !isStatus && !isTrackingIssue && !isPreventStale && !isDifficulty;
 }
 
+// Get the issue number from an issue link in the forms `<keyword> <issue url>` or `<keyword> #<issue number>`.
+function getIssueLink (repoUrl, body) {
+    const urlPattern = new RegExp(`(close|closes|closed|fix|fixes|fixed|resolve|resolves|resolved) ${repoUrl}/issues/(?<issue_number>\\d+)`, 'i')
+    const issuePattern = new RegExp(`(close|closes|closed|fix|fixes|fixed|resolve|resolves|resolved) \#(?<issue_number>\\d+)`, 'i')
+
+    const urlRe = body.match(urlPattern);
+    const issueRe = body.match(issuePattern);
+    if (urlRe?.groups?.issue_number) {
+        return urlRe.groups.issue_number
+    } else {
+        return issueRe?.groups?.issue_number
+    }
+}
+
 module.exports = async ({ github, context }) => {
     try {
         const prNumber = context.payload.pull_request.number;
         const prBody = context.payload.pull_request.body;
         const repo = context.repo;
 
         const repoUrl = context.payload.repository.html_url;
-        const pattern = new RegExp(`(close|closes|closed|fix|fixes|fixed|resolve|resolves|resolved) ${repoUrl}/issues/(?<issue_number>\\d+)`, 'i')
-
-        const re = prBody.match(pattern);
-        const issueNumber = re?.groups?.issue_number;
-
+        const issueNumber = getIssueLink(repoUrl, prBody);
         if (!issueNumber) {
             console.log('No issue reference found in PR description.');
             return;

diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml
@@ -11,8 +11,10 @@ on:
 env:
   REPO_NAME: ${{ github.repository_owner }}/reth
   IMAGE_NAME: ${{ github.repository_owner }}/reth
+  OP_IMAGE_NAME: ${{ github.repository_owner }}/op-reth
   CARGO_TERM_COLOR: always
   DOCKER_IMAGE_NAME: ghcr.io/${{ github.repository_owner }}/reth
+  OP_DOCKER_IMAGE_NAME: ghcr.io/${{ github.repository_owner }}/op-reth
   DOCKER_USERNAME: ${{ github.actor }}
 
 jobs:
@@ -36,9 +38,15 @@ jobs:
         run: |
           docker run --privileged --rm tonistiigi/binfmt --install arm64,amd64
           docker buildx create --use --name cross-builder
-      - name: Build and push image, tag as "latest"
+      - name: Build and push reth image, tag as "latest"
         if: ${{ contains(github.event.ref, 'beta') }}
         run: make PROFILE=maxperf docker-build-push-latest
-      - name: Build and push image
+      - name: Build and push reth image
         if: ${{ ! contains(github.event.ref, 'beta') }}
         run: make PROFILE=maxperf docker-build-push
+      - name: Build and push op-reth image, tag as "latest"
+        if: ${{ contains(github.event.ref, 'beta') }}
+        run: make IMAGE_NAME=$OP_IMAGE_NAME DOCKER_IMAGE_NAME=$OP_DOCKER_IMAGE_NAME PROFILE=maxperf op-docker-build-push-latest
+      - name: Build and push op-reth image
+        if: ${{ ! contains(github.event.ref, 'beta') }}
+        run: make IMAGE_NAME=$OP_IMAGE_NAME DOCKER_IMAGE_NAME=$OP_DOCKER_IMAGE_NAME PROFILE=maxperf op-docker-build-push
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -27,8 +27,7 @@ jobs:
       - uses: Swatinem/rust-cache@v2
         with:
           cache-on-failure: true
-      - run:
-          cargo clippy --bin "${{ matrix.binary }}" --workspace --features "${{ matrix.network }} asm-keccak jemalloc jemalloc-prof min-error-logs min-warn-logs min-info-logs min-debug-logs min-trace-logs"
+      - run: cargo clippy --bin "${{ matrix.binary }}" --workspace --features "${{ matrix.network }} asm-keccak jemalloc jemalloc-prof min-error-logs min-warn-logs min-info-logs min-debug-logs min-trace-logs"
         env:
           RUSTFLAGS: -D warnings
 
@@ -95,9 +94,7 @@ jobs:
         env:
           # Keep in sync with ./book.yml:jobs.build
           # This should only add `-D warnings`
-          RUSTDOCFLAGS:
-            --cfg docsrs --show-type-layout --generate-link-to-definition --enable-index-page
-            -Zunstable-options -D warnings
+          RUSTDOCFLAGS: --cfg docsrs --show-type-layout --generate-link-to-definition --enable-index-page -Zunstable-options -D warnings
 
   fmt:
     name: fmt
@@ -110,6 +107,12 @@ jobs:
           components: rustfmt
       - run: cargo fmt --all --check
 
+  codespell:
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+    steps:
+      - uses: codespell-project/actions-codespell@v2
+
   grafana:
     runs-on: ubuntu-latest
     timeout-minutes: 30
@@ -124,7 +127,7 @@ jobs:
     name: lint success
     runs-on: ubuntu-latest
     if: always()
-    needs: [clippy-binaries, clippy, crate-checks, docs, fmt, grafana]
+    needs: [clippy-binaries, clippy, crate-checks, docs, fmt, codespell, grafana]
     timeout-minutes: 30
     steps:
       - name: Decide whether the needed jobs succeeded or failed

diff --git a/.github/workflows/release-dist.yml b/.github/workflows/release-dist.yml
@@ -14,7 +14,7 @@ jobs:
       - name: Update Homebrew formula
         uses: dawidd6/action-homebrew-bump-formula@v3
         with:
-          token: ${{ secrets.GITHUB_TOKEN }}
+          token: ${{ secrets.HOMEBREW }}
           no_fork: true
           tap: paradigmxyz/brew
           formula: reth
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -10,6 +10,7 @@ on:
 
 env:
   REPO_NAME: ${{ github.repository_owner }}/reth
+  OP_IMAGE_NAME: ${{ github.repository_owner }}/op-reth
   IMAGE_NAME: ${{ github.repository_owner }}/reth
   CARGO_TERM_COLOR: always
 
@@ -26,11 +27,11 @@ jobs:
 
   build:
     name: build release
-    runs-on: ${{ matrix.os }}
+    runs-on: ${{ matrix.configs.os }}
     needs: extract-version
     strategy:
       matrix:
-        include:
+        configs:
           - target: x86_64-unknown-linux-gnu
             os: ubuntu-20.04
             profile: maxperf
@@ -46,29 +47,34 @@ jobs:
           - target: x86_64-pc-windows-gnu
             os: ubuntu-20.04
             profile: maxperf
+        build:
+          - command: build
+            binary: reth
+          - command: op-build
+            binary: op-reth
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
         with:
-          target: ${{ matrix.target }}
+          target: ${{ matrix.configs.target }}
       - uses: taiki-e/install-action@cross
       - uses: Swatinem/rust-cache@v2
         with:
           cache-on-failure: true
 
       - name: Apple M1 setup
-        if: matrix.target == 'aarch64-apple-darwin'
+        if: matrix.configs.target == 'aarch64-apple-darwin'
         run: |
           echo "SDKROOT=$(xcrun -sdk macosx --show-sdk-path)" >> $GITHUB_ENV
-          echo "MACOSX_DEPLOYMENT_TARGET=$(xcrun -sdk macosx --show-sdk-os-version)" >> $GITHUB_ENV
+          echo "MACOSX_DEPLOYMENT_TARGET=$(xcrun -sdk macosx --show-sdk-platform-version)" >> $GITHUB_ENV
 
       - name: Build Reth
-        run: make PROFILE=${{ matrix.profile }} build-${{ matrix.target }}
+        run: make PROFILE=${{ matrix.configs.profile }} ${{ matrix.build.command }}-${{ matrix.configs.target }}
       - name: Move binary
         run: |
           mkdir artifacts
-          [[ "${{ matrix.target }}" == *windows* ]] && ext=".exe"
-          mv "target/${{ matrix.target }}/${{ matrix.profile }}/reth${ext}" ./artifacts
+          [[ "${{ matrix.configs.target }}" == *windows* ]] && ext=".exe"
+          mv "target/${{ matrix.configs.target }}/${{ matrix.configs.profile }}/${{ matrix.build.binary }}${ext}" ./artifacts
 
       - name: Configure GPG and create artifacts
         env:
@@ -78,22 +84,22 @@ jobs:
           export GPG_TTY=$(tty)
           echo -n "$GPG_SIGNING_KEY" | base64 --decode | gpg --batch --import
           cd artifacts
-          tar -czf reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz reth*
-          echo "$GPG_PASSPHRASE" | gpg --passphrase-fd 0 --pinentry-mode loopback --batch -ab reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz
+          tar -czf ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz ${{ matrix.build.binary }}*
+          echo "$GPG_PASSPHRASE" | gpg --passphrase-fd 0 --pinentry-mode loopback --batch -ab ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz
           mv *tar.gz* ..
         shell: bash
 
       - name: Upload artifact
         uses: actions/upload-artifact@v4
         with:
-          name: reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz
-          path: reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz
+          name: ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz
+          path: ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz
 
       - name: Upload signature
         uses: actions/upload-artifact@v4
         with:
-          name: reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz.asc
-          path: reth-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.target }}.tar.gz.asc
+          name: ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz.asc
+          path: ${{ matrix.build.binary }}-${{ needs.extract-version.outputs.VERSION }}-${{ matrix.configs.target }}.tar.gz.asc
 
   draft-release:
     name: draft release
@@ -184,7 +190,7 @@ jobs:
           ENDBODY
           )
           assets=()
-          for asset in ./reth-*.tar.gz*; do
+          for asset in ./*reth-*.tar.gz*; do
               assets+=("$asset/$asset")
           done
           tag_name="${{ env.VERSION }}"

diff --git a/.github/workflows/unit.yml b/.github/workflows/unit.yml
@@ -57,6 +57,7 @@ jobs:
         uses: actions/checkout@v4
         with:
           repository: ethereum/tests
+          ref: 1c23e3c27ac53b794de0844d2d5e19cd2495b9d8
           path: testing/ef-tests/ethereum-tests
           submodules: recursive
           fetch-depth: 1

diff --git a/CODEOWNERS b/CODEOWNERS
@@ -1,24 +1,27 @@
 *                           @gakonst
 bin/                        @onbjerg
 crates/blockchain-tree      @rakita @rkrasiuk
-crates/consensus/auto-seal  @mattsse
-crates/consensus/beacon     @rkrasiuk @mattsse @Rjected
+crates/cli/                 @onbjerg @mattsse
+crates/consensus            @rkrasiuk @mattsse @Rjected
 crates/exex                 @onbjerg @shekhirin
 crates/metrics              @onbjerg
 crates/net/                 @emhane @mattsse @Rjected
 crates/net/downloaders/     @onbjerg @rkrasiuk
+crates/node/                @mattsse @Rjected @onbjerg
+crates/node-core/           @mattsse @Rjected @onbjerg
+crates/node-ethereum/       @mattsse @Rjected
 crates/payload/             @mattsse @Rjected
 crates/prune                @shekhirin @joshieDo
-crates/revm/src/            @rakita
-crates/revm/                @mattsse
+crates/revm/                @mattsse @rakita
 crates/rpc/                 @mattsse @Rjected
-crates/rpc/rpc-types        @mattsse @Rjected @Evalir
-crates/rpc/rpc-types-compat @mattsse @Rjected @Evalir
 crates/stages/              @onbjerg @rkrasiuk @shekhirin
+crates/stages-api/          @onbjerg @rkrasiuk @shekhirin
 crates/static-file          @joshieDo @shekhirin
 crates/storage/             @rakita @joshieDo @shekhirin
 crates/tasks                @mattsse
 crates/tracing              @onbjerg
 crates/transaction-pool/    @mattsse
 crates/trie                 @rkrasiuk
+crates/trie-parallel        @rkrasiuk
+crates/optimism             @mattsse
 .github/                    @onbjerg @gakonst @DaniPopes