feat(l1, l1): make levm the default vm. (#2603)

**Motivation**
Levm is becoming more mature, and it needed for the L2. Let's set it as
default across the board.

Author: mpaulucci

.github/workflows/common_hive_reports.yaml

@@ -12,9 +12,6 @@ on:
         required: true
         type: string
 
-env:
-  CARGO_TERM_COLOR: always
-
 jobs:
   run-hive:
     name: Hive (${{ inputs.evm }}) - ${{ matrix.test.name }}

.github/workflows/pr-main_l1.yaml

@@ -6,7 +6,6 @@ on:
   pull_request:
     branches: ["**"]
     paths-ignore:
-      - "crates/vm/levm/**" # We run this in a separate workflow
       - "crates/l2/**" # Behind a feature flag not used in this workflow
 
 concurrency:
@@ -194,7 +193,6 @@ jobs:
         run: |
           docker load --input /tmp/ethrex_image.tar
 
-      # By default ethrex uses revm as evm backend.
       - name: Run Hive Simulation
         run: chmod +x hive && ./hive --client ethrex --ethrex.flags "${{ matrix.ethrex_flags }}" --sim ${{ matrix.simulation }} --sim.limit "${{ matrix.test_pattern }}" --sim.parallelism 16
 

.github/workflows/pr-main_levm.yaml

@@ -17,9 +17,6 @@ concurrency:
 permissions:
   pull-requests: write
 
-env:
-  CARGO_TERM_COLOR: always
-
 jobs:
   ef-test:
     if: ${{ github.event_name != 'merge_group' }}
@@ -199,86 +196,17 @@ jobs:
             Same results between main branch and the current PR.
           edit-mode: replace
 
-  hive-report-creation:
-    uses: ./.github/workflows/common_hive_reports.yaml
-    with:
-      evm: levm
-      job_type: trigger
-
-  hive-report-creation-main:
-    uses: ./.github/workflows/common_hive_reports.yaml
-    with:
-      evm: levm
-      job_type: main
-
-  # Check we don't have a regression against main
-  hive-test:
-    if: ${{ github.event_name != 'merge_group' }}
-    name: Hive Tests Check
-    needs: [hive-report-creation, hive-report-creation-main]
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout sources
-        uses: actions/checkout@v4
-
-      - name: Download results (pr)
-        uses: actions/download-artifact@v4
-        with:
-          name: results_levm_trigger.md
-
-      - name: Rename result (1)
-        run: |
-          cp results.md results_levm.md
-          echo "RESULTS:"
-          cat results_levm.md
-
-      - name: Download results (main)
-        uses: actions/download-artifact@v4
-        with:
-          name: results_levm_main.md
-        # If we don't have the file it will use the result(1) file
-        continue-on-error: true
-
-      - name: Rename result (2)
-        run: |
-          cp results.md results_levm_main.md
-          echo "RESULTS:"
-          cat results_levm_main.md
-
-      - name: Create diff message
-        run: |
-          bash .github/scripts/hive_levm_revm_diff.sh results_levm_main.md results_levm.md >> diff.md
-          cat diff.md >> $GITHUB_STEP_SUMMARY
-          echo "SUMMARY:"
-          cat diff.md
-
-      - name: Check Regression
-        run: |
-          if grep -q "No differences found" diff.md; then
-            echo "No differences found."
-          elif ! grep -q "regression" diff.md; then
-            echo "No regression found."
-          else
-            echo "Differences found."
-            exit 1
-          fi
-
   # The purpose of this job is to add it as a required check in GitHub so that we don't have to add every individual job as a required check
   all-tests:
     # "Integration Test" is a required check, don't change the name
     name: Integration Test
     runs-on: ubuntu-latest
-    needs: [hive-test, ef-test]
+    needs: [ef-test]
     # Make sure this job runs even if the previous jobs failed or were skipped
-    if: ${{ always() && needs.hive-test.result != 'skipped' && needs.ef-test.result != 'skipped' }}
+    if: ${{ always() && needs.ef-test.result != 'skipped' }}
     steps:
       - name: Check if any job failed
         run: |
-          if [ "${{ needs.hive-test.result }}" != "success" ]; then
-            echo "Job Hive Tests Check failed"
-            exit 1
-          fi
-
           if [ "${{ needs.ef-test.result }}" != "success" ]; then
             echo "Job EF Tests Check failed"
             exit 1
@@ -330,38 +258,3 @@ jobs:
         run: |
           cd crates/vm/levm
           make revm-comparison
-  lint:
-    # "Lint" is a required check, don't change the name
-    name: Lint
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout sources
-        uses: actions/checkout@v4
-
-      - name: Rustup toolchain install
-        uses: dtolnay/rust-toolchain@stable
-        with:
-          components: rustfmt, clippy
-
-      - name: Add Rust Cache
-        uses: Swatinem/rust-cache@v2
-
-      - name: Run cargo check
-        run: |
-          cd crates/vm/levm
-          cargo check
-
-      - name: Run cargo build
-        run: |
-          cd crates/vm/levm
-          cargo build
-
-      - name: Run cargo clippy
-        run: |
-          cd crates/vm/levm
-          cargo clippy --all-targets --all-features -- -D warnings
-
-      - name: Run cargo fmt
-        run: |
-          cd crates/vm/levm
-          cargo fmt --all -- --check

README.md

@@ -235,7 +235,7 @@ Node options:
           Has to be `levm` or `revm`
 
           [env: ETHREX_EVM=]
-          [default: revm]
+          [default: levm]
 
       --log.level <LOG_LEVEL>
           Possible values: info, debug, trace, warn, error
@@ -248,11 +248,11 @@ P2P options:
 
       --syncmode <SYNC_MODE>
           Can be either "full" or "snap" with "full" as default value.
-          
+
           [default: full]
 
       --p2p.enabled
-          
+
 
       --p2p.addr <ADDRESS>
           [default: 0.0.0.0]
@@ -262,12 +262,12 @@ P2P options:
 
       --discovery.addr <ADDRESS>
           UDP address for P2P discovery.
-          
+
           [default: 0.0.0.0]
 
       --discovery.port <PORT>
           UDP port for P2P discovery.
-          
+
           [default: 30303]
 
 RPC options:
@@ -285,17 +285,17 @@ RPC options:
 
       --authrpc.addr <ADDRESS>
           Listening address for the authenticated rpc server.
-          
+
           [default: localhost]
 
       --authrpc.port <PORT>
           Listening port for the authenticated rpc server.
-          
+
           [default: 8551]
 
       --authrpc.jwtsecret <JWTSECRET_PATH>
           Receives the jwt secret used for authenticated rpc requests.
-          
+
           [default: jwt.hex]
 ```
 <!-- END_CLI_HELP -->

cmd/ethrex/cli.rs

@@ -97,7 +97,7 @@ pub struct Options {
     pub dev: bool,
     #[arg(
         long = "evm",
-        default_value = "revm",
+        default_value = "levm",
         value_name = "EVM_BACKEND",
         help = "Has to be `levm` or `revm`",
         value_parser = utils::parse_evm_engine,