Added bail to stop execution when we have an error during deployment

.github/workflows/test.yml

@@ -99,28 +99,9 @@ jobs:
       - name: Install Geth on Ubuntu
         if: matrix.os == 'ubuntu-24.04'
         run: |
+          sudo add-apt-repository -y ppa:ethereum/ethereum
           sudo apt-get update
-          sudo apt-get install -y protobuf-compiler
+          sudo apt-get install -y ethereum protobuf-compiler
-
-          # We were facing some issues in CI with the 1.16.* versions of geth, and specifically on
-          # Ubuntu. Eventually, we found out that the last version of geth that worked in our CI was
-          # version 1.15.11. Thus, this is the version that we want to use in CI. The PPA sadly does
-          # not have historic versions of Geth and therefore we need to resort to downloading pre
-          # built binaries for Geth and the surrounding tools which is what the following parts of 
-          # the script do.
-
-          sudo apt-get install -y wget ca-certificates tar
-          ARCH=$(uname -m)
-          if [ "$ARCH" = "x86_64" ]; then
-            URL="https://gethstore.blob.core.windows.net/builds/geth-alltools-linux-amd64-1.15.11-36b2371c.tar.gz"
-          elif [ "$ARCH" = "aarch64" ]; then
-            URL="https://gethstore.blob.core.windows.net/builds/geth-alltools-linux-arm64-1.15.11-36b2371c.tar.gz"
-          else
-            echo "Unsupported architecture: $ARCH"
-            exit 1
-          fi
-          wget -qO- "$URL" | sudo tar xz -C /usr/local/bin --strip-components=1
-          geth --version
 
       - name: Install Geth on macOS
         if: matrix.os == 'macos-14'

Cargo.lock

@@ -2071,6 +2071,29 @@ dependencies = [
  "syn 2.0.101",
 ]
 
+[[package]]
+name = "env_filter"
+version = "0.1.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "186e05a59d4c50738528153b83b0b0194d3a29507dfec16eccd4b342903397d0"
+dependencies = [
+ "log",
+ "regex",
+]
+
+[[package]]
+name = "env_logger"
+version = "0.11.8"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "13c863f0904021b108aa8b2f55046443e6b1ebde8fd4a15c399893aae4fa069f"
+dependencies = [
+ "anstream",
+ "anstyle",
+ "env_filter",
+ "jiff",
+ "log",
+]
+
 [[package]]
 name = "environmental"
 version = "1.1.4"
@@ -2939,6 +2962,30 @@ version = "1.0.15"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "4a5f13b858c8d314ee3e8f639011f7ccefe71f97f96e50151fb991f267928e2c"
 
+[[package]]
+name = "jiff"
+version = "0.2.14"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a194df1107f33c79f4f93d02c80798520551949d59dfad22b6157048a88cca93"
+dependencies = [
+ "jiff-static",
+ "log",
+ "portable-atomic",
+ "portable-atomic-util",
+ "serde",
+]
+
+[[package]]
+name = "jiff-static"
+version = "0.2.14"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "6c6e1db7ed32c6c71b759497fae34bf7933636f75a251b9e736555da426f6442"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn 2.0.101",
+]
+
 [[package]]
 name = "js-sys"
 version = "0.3.77"
@@ -3550,6 +3597,21 @@ dependencies = [
  "syn 2.0.101",
 ]
 
+[[package]]
+name = "portable-atomic"
+version = "1.11.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "350e9b48cbc6b0e028b0473b114454c6316e57336ee184ceab6e53f72c178b3e"
+
+[[package]]
+name = "portable-atomic-util"
+version = "0.2.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d8a2f0d8d040d7848a709caf78912debcc3f33ee4b3cac47d73d1e1069e83507"
+dependencies = [
+ "portable-atomic",
+]
+
 [[package]]
 name = "potential_utf"
 version = "0.1.2"
@@ -3935,13 +3997,13 @@ name = "revive-dt-compiler"
 version = "0.1.0"
 dependencies = [
  "anyhow",
+ "log",
  "revive-common",
  "revive-dt-config",
  "revive-dt-solc-binaries",
  "revive-solc-json-interface",
  "semver 1.0.26",
  "serde_json",
- "tracing",
 ]
 
 [[package]]
@@ -3962,6 +4024,8 @@ dependencies = [
  "alloy",
  "anyhow",
  "clap",
+ "env_logger",
+ "log",
  "rayon",
  "revive-dt-compiler",
  "revive-dt-config",
@@ -3972,8 +4036,6 @@ dependencies = [
  "revive-solc-json-interface",
  "serde_json",
  "temp-dir",
- "tracing",
- "tracing-subscriber",
 ]
 
 [[package]]
@@ -3984,10 +4046,10 @@ dependencies = [
  "alloy-primitives",
  "alloy-sol-types",
  "anyhow",
+ "log",
  "semver 1.0.26",
  "serde",
  "serde_json",
- "tracing",
 ]
 
 [[package]]
@@ -3996,13 +4058,13 @@ version = "0.1.0"
 dependencies = [
  "alloy",
  "anyhow",
+ "log",
  "revive-dt-config",
  "revive-dt-node-interaction",
  "serde_json",
  "sp-core",
  "sp-runtime",
  "temp-dir",
- "tracing",
 ]
 
 [[package]]
@@ -4011,9 +4073,9 @@ version = "0.1.0"
 dependencies = [
  "alloy",
  "anyhow",
+ "log",
  "once_cell",
  "tokio",
- "tracing",
 ]
 
 [[package]]
@@ -4021,12 +4083,12 @@ name = "revive-dt-report"
 version = "0.1.0"
 dependencies = [
  "anyhow",
+ "log",
  "revive-dt-config",
  "revive-dt-format",
  "revive-solc-json-interface",
  "serde",
  "serde_json",
- "tracing",
 ]
 
 [[package]]
@@ -4035,11 +4097,11 @@ version = "0.1.0"
 dependencies = [
  "anyhow",
  "hex",
+ "log",
  "reqwest",
  "semver 1.0.26",
  "serde",
  "sha2 0.10.9",
- "tracing",
 ]
 
 [[package]]
@@ -5507,16 +5569,6 @@ dependencies = [
  "tracing-core",
 ]
 
-[[package]]
-name = "tracing-serde"
-version = "0.2.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "704b1aeb7be0d0a84fc9828cae51dab5970fee5088f83d1dd7ee6f6246fc6ff1"
-dependencies = [
- "serde",
- "tracing-core",
-]
-
 [[package]]
 name = "tracing-subscriber"
 version = "0.3.19"
@@ -5527,8 +5579,6 @@ dependencies = [
  "nu-ansi-term",
  "once_cell",
  "regex",
- "serde",
- "serde_json",
  "sharded-slab",
  "smallvec",
  "thread_local",
@@ -5536,7 +5586,6 @@ dependencies = [
  "tracing",
  "tracing-core",
  "tracing-log",
- "tracing-serde",
 ]
 
 [[package]]

Cargo.toml

@@ -4,7 +4,9 @@ members = ["crates/*"]
 
 [workspace.package]
 version = "0.1.0"
-authors = ["Parity Technologies <admin@parity.io>"]
+authors = [
+    "Parity Technologies <admin@parity.io>",
+]
 license = "MIT/Apache-2.0"
 edition = "2024"
 repository = "https://github.com/paritytech/revive-differential-testing.git"
@@ -25,31 +27,22 @@ alloy-primitives = "1.2.1"
 alloy-sol-types = "1.2.1"
 anyhow = "1.0"
 clap = { version = "4", features = ["derive"] }
+env_logger = "0.11.8"
 hex = "0.4.3"
 reqwest = { version = "0.12.15", features = ["blocking", "json"] }
+log = "0.4.27"
 once_cell = "1.21"
 rayon = { version = "1.10" }
 semver = { version = "1.0", features = ["serde"] }
 serde = { version = "1.0", default-features = false, features = ["derive"] }
-serde_json = { version = "1.0", default-features = false, features = [
+serde_json = { version = "1.0", default-features = false, features = ["arbitrary_precision", "std"] }
-    "arbitrary_precision",
-    "std",
-] }
 sha2 = { version = "0.10.9" }
 sp-core = "36.1.0"
 sp-runtime = "41.1.0"
 temp-dir = { version = "0.1.16" }
 tempfile = "3.3"
-tokio = { version = "1", default-features = false, features = [
+tokio = { version = "1", default-features = false, features = ["rt-multi-thread"] }
-    "rt-multi-thread",
-] }
 uuid = { version = "1.8", features = ["v4"] }
-tracing = "0.1.41"
-tracing-subscriber = { version = "0.3.19", default-features = false, features = [
-    "fmt",
-    "json",
-    "env-filter",
-] }
 
 # revive compiler
 revive-solc-json-interface = { git = "https://github.com/paritytech/revive", rev = "3389865af7c3ff6f29a586d82157e8bc573c1a8e" }

crates/compiler/Cargo.toml

@@ -16,4 +16,4 @@ revive-dt-solc-binaries = { workspace = true }
 revive-common = { workspace = true }
 semver = { workspace = true }
 serde_json = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }

crates/compiler/src/revive_resolc.rs

@@ -42,7 +42,7 @@ impl SolidityCompiler for Resolc {
 
         if !output.status.success() {
             let message = String::from_utf8_lossy(&stderr);
-            tracing::error!(
+            log::error!(
                 "resolc failed exit={} stderr={} JSON-in={} ",
                 output.status,
                 &message,

crates/compiler/src/solc.rs

@@ -34,7 +34,7 @@ impl SolidityCompiler for Solc {
 
         if !output.status.success() {
             let message = String::from_utf8_lossy(&output.stderr);
-            tracing::error!("solc failed exit={} stderr={}", output.status, &message);
+            log::error!("solc failed exit={} stderr={}", output.status, &message);
             return Ok(CompilerOutput {
                 input,
                 output: Default::default(),

crates/core/Cargo.toml

@@ -23,8 +23,8 @@ revive-dt-report = { workspace = true }
 alloy = { workspace = true }
 anyhow = { workspace = true }
 clap = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
-tracing-subscriber = { workspace = true }
+env_logger = { workspace = true }
 rayon = { workspace = true }
 revive-solc-json-interface = { workspace = true }
 serde_json = { workspace = true }

crates/core/src/driver/mod.rs

@@ -19,7 +19,6 @@ use revive_dt_report::reporter::{CompilationTask, Report, Span};
 use revive_solc_json_interface::SolcStandardJsonOutput;
 use serde_json::Value;
 use std::collections::HashMap as StdHashMap;
-use tracing::Level;
 
 use crate::Platform;
 
@@ -74,7 +73,7 @@ where
             .solc_optimizer(mode.solc_optimize());
 
         for (file, _contract) in metadata.contract_sources()?.values() {
-            tracing::debug!("contract source {}", file.display());
+            log::debug!("contract source {}", file.display());
             compiler = compiler.with_source(file)?;
         }
 
@@ -97,13 +96,11 @@ where
                     if let Some(contracts) = &last_output.contracts {
                         for (file, contracts_map) in contracts {
                             for contract_name in contracts_map.keys() {
-                                tracing::debug!(
+                                log::debug!("Compiled contract: {contract_name} from file: {file}");
-                                    "Compiled contract: {contract_name} from file: {file}"
-                                );
                             }
                         }
                     } else {
-                        tracing::warn!("Compiled contracts field is None");
+                        log::warn!("Compiled contracts field is None");
                     }
                 }
 
@@ -111,7 +108,7 @@ where
                 Ok(())
             }
             Err(error) => {
-                tracing::error!("Failed to compile contract: {:?}", error.to_string());
+                log::error!("Failed to compile contract: {:?}", error.to_string());
                 task.error = Some(error.to_string());
                 Err(error)
             }
@@ -123,11 +120,11 @@ where
         input: &Input,
         node: &T::Blockchain,
     ) -> anyhow::Result<(TransactionReceipt, GethTrace, DiffMode)> {
-        tracing::trace!("Calling execute_input for input: {input:?}");
+        log::trace!("Calling execute_input for input: {input:?}");
 
         let nonce = node.fetch_add_nonce(input.caller)?;
 
-        tracing::debug!(
+        log::debug!(
             "Nonce calculated on the execute contract, calculated nonce {}, for contract {}, having address {} on node: {}",
             &nonce,
             &input.instance,
@@ -142,21 +139,21 @@ where
             &self.deployed_abis,
         ) {
             Ok(tx) => {
-                tracing::debug!("Legacy transaction data: {tx:#?}");
+                log::debug!("Legacy transaction data: {tx:#?}");
                 tx
             }
             Err(err) => {
-                tracing::error!("Failed to construct legacy transaction: {err:?}");
+                log::error!("Failed to construct legacy transaction: {err:?}");
                 return Err(err);
             }
         };
 
-        tracing::trace!("Executing transaction for input: {input:?}");
+        log::trace!("Executing transaction for input: {input:?}");
 
         let receipt = match node.execute_transaction(tx) {
             Ok(receipt) => receipt,
             Err(err) => {
-                tracing::error!(
+                log::error!(
                     "Failed to execute transaction when executing the contract: {}, {:?}",
                     &input.instance,
                     err
@@ -165,14 +162,14 @@ where
             }
         };
 
-        tracing::trace!(
+        log::trace!(
             "Transaction receipt for executed contract: {} - {:?}",
             &input.instance,
             receipt,
         );
 
         let trace = node.trace_transaction(receipt.clone())?;
-        tracing::trace!(
+        log::trace!(
             "Trace result for contract: {} - {:?}",
             &input.instance,
             trace
@@ -184,7 +181,7 @@ where
     }
 
     pub fn deploy_contracts(&mut self, input: &Input, node: &T::Blockchain) -> anyhow::Result<()> {
-        tracing::debug!(
+        log::debug!(
             "Deploying contracts {}, having address {} on node: {}",
             &input.instance,
             &input.caller,
@@ -192,7 +189,7 @@ where
         );
         for output in self.contracts.values() {
             let Some(contract_map) = &output.contracts else {
-                tracing::debug!(
+                log::debug!(
                     "No contracts in output — skipping deployment for this input {}",
                     &input.instance
                 );
@@ -201,7 +198,7 @@ where
 
             for contracts in contract_map.values() {
                 for (contract_name, contract) in contracts {
-                    tracing::debug!(
+                    log::debug!(
                         "Contract name is: {:?} and the input name is: {:?}",
                         &contract_name,
                         &input.instance
@@ -214,13 +211,13 @@ where
                         .map(|b| b.object.clone());
 
                     let Some(code) = bytecode else {
-                        tracing::error!("no bytecode for contract {contract_name}");
+                        log::error!("no bytecode for contract {contract_name}");
                         continue;
                     };
 
                     let nonce = node.fetch_add_nonce(input.caller)?;
 
-                    tracing::debug!(
+                    log::debug!(
                         "Calculated nonce {}, for contract {}, having address {} on node: {}",
                         &nonce,
                         &input.instance,
@@ -242,7 +239,7 @@ where
                     let receipt = match node.execute_transaction(tx) {
                         Ok(receipt) => receipt,
                         Err(err) => {
-                            tracing::error!(
+                            log::error!(
                                 "Failed to execute transaction when deploying the contract on node : {:?}, {:?}, {:?}",
                                 std::any::type_name::<T>(),
                                 &contract_name,
@@ -252,7 +249,7 @@ where
                         }
                     };
 
-                    tracing::debug!(
+                    log::debug!(
                         "Deployment tx sent for {} with nonce {} → tx hash: {:?}, on node: {:?}",
                         contract_name,
                         nonce,
@@ -260,7 +257,7 @@ where
                         std::any::type_name::<T>(),
                     );
 
-                    tracing::trace!(
+                    log::trace!(
                         "Deployed transaction receipt for contract: {} - {:?}, on node: {:?}",
                         &contract_name,
                         receipt,
@@ -268,7 +265,7 @@ where
                     );
 
                     let Some(address) = receipt.contract_address else {
-                        tracing::error!(
+                        log::error!(
                             "contract {contract_name} deployment did not return an address"
                         );
                         continue;
@@ -276,7 +273,7 @@ where
 
                     self.deployed_contracts
                         .insert(contract_name.clone(), address);
-                    tracing::trace!(
+                    log::trace!(
                         "deployed contract `{}` at {:?}, on node {:?}",
                         contract_name,
                         address,
@@ -284,7 +281,7 @@ where
                     );
 
                     if let Some(Value::String(metadata_json_str)) = &contract.metadata {
-                        tracing::trace!(
+                        log::trace!(
                             "metadata found for contract {contract_name}, {metadata_json_str}"
                         );
 
@@ -295,7 +292,7 @@ where
                                 {
                                     match serde_json::from_value::<JsonAbi>(abi_value.clone()) {
                                         Ok(parsed_abi) => {
-                                            tracing::trace!(
+                                            log::trace!(
                                                 "ABI found in metadata for contract {}",
                                                 &contract_name
                                             );
@@ -332,7 +329,7 @@ where
             }
         }
 
-        tracing::debug!("Available contracts: {:?}", self.deployed_contracts.keys());
+        log::debug!("Available contracts: {:?}", self.deployed_contracts.keys());
 
         Ok(())
     }
@@ -365,28 +362,28 @@ where
     }
 
     pub fn trace_diff_mode(label: &str, diff: &DiffMode) {
-        tracing::trace!("{label} - PRE STATE:");
+        log::trace!("{label} - PRE STATE:");
         for (addr, state) in &diff.pre {
             Self::trace_account_state("  [pre]", addr, state);
         }
 
-        tracing::trace!("{label} - POST STATE:");
+        log::trace!("{label} - POST STATE:");
         for (addr, state) in &diff.post {
             Self::trace_account_state("  [post]", addr, state);
         }
     }
 
     fn trace_account_state(prefix: &str, addr: &Address, state: &AccountState) {
-        tracing::trace!("{prefix} 0x{addr:x}");
+        log::trace!("{prefix} 0x{addr:x}");
 
         if let Some(balance) = &state.balance {
-            tracing::trace!("{prefix}   balance: {balance}");
+            log::trace!("{prefix}   balance: {balance}");
         }
         if let Some(nonce) = &state.nonce {
-            tracing::trace!("{prefix}   nonce: {nonce}");
+            log::trace!("{prefix}   nonce: {nonce}");
         }
         if let Some(code) = &state.code {
-            tracing::trace!("{prefix}   code: {code}");
+            log::trace!("{prefix}   code: {code}");
         }
     }
 
@@ -398,75 +395,61 @@ where
             let mut follower_state = State::<F>::new(self.config, span);
             follower_state.build_contracts(&mode, self.metadata)?;
 
-            for (case_idx, case) in self.metadata.cases.iter().enumerate() {
+            for case in &self.metadata.cases {
-                // Creating a tracing span to know which case within the metadata is being executed
-                // and which one we're getting logs for.
-                let tracing_span = tracing::span!(
-                    Level::INFO,
-                    "Executing case",
-                    case = case.name,
-                    case_idx = case_idx
-                );
-                let _guard = tracing_span.enter();
-
                 for input in &case.inputs {
-                    tracing::debug!("Starting deploying contract {}", &input.instance);
+                    log::debug!("Starting deploying contract {}", &input.instance);
                     if let Err(err) = leader_state.deploy_contracts(input, self.leader_node) {
-                        tracing::error!("Leader deployment failed for {}: {err}", input.instance);
+                        log::error!("Leader deployment failed for {}: {err}", input.instance);
                         continue;
                     } else {
-                        tracing::debug!("Leader deployment succeeded for {}", &input.instance);
+                        log::debug!("Leader deployment succeeded for {}", &input.instance);
                     }
 
                     if let Err(err) = follower_state.deploy_contracts(input, self.follower_node) {
-                        tracing::error!("Follower deployment failed for {}: {err}", input.instance);
+                        log::error!("Follower deployment failed for {}: {err}", input.instance);
                         continue;
                     } else {
-                        tracing::debug!("Follower deployment succeeded for {}", &input.instance);
+                        log::debug!("Follower deployment succeeded for {}", &input.instance);
                     }
 
-                    tracing::debug!("Starting executing contract {}", &input.instance);
+                    log::debug!("Starting executing contract {}", &input.instance);
 
-                    let (leader_receipt, _, leader_diff) =
+                    let (leader_receipt, _, leader_diff) = match leader_state
-                        match leader_state.execute_input(input, self.leader_node) {
+                        .execute_input(input, self.leader_node)
-                            Ok(result) => result,
+                    {
-                            Err(err) => {
+                        Ok(result) => result,
-                                tracing::error!(
+                        Err(err) => {
-                                    "Leader execution failed for {}: {err}",
+                            log::error!("Leader execution failed for {}: {err}", input.instance);
-                                    input.instance
+                            continue;
-                                );
+                        }
-                                continue;
+                    };
-                            }
-                        };
 
-                    let (follower_receipt, _, follower_diff) =
+                    let (follower_receipt, _, follower_diff) = match follower_state
-                        match follower_state.execute_input(input, self.follower_node) {
+                        .execute_input(input, self.follower_node)
-                            Ok(result) => result,
+                    {
-                            Err(err) => {
+                        Ok(result) => result,
-                                tracing::error!(
+                        Err(err) => {
-                                    "Follower execution failed for {}: {err}",
+                            log::error!("Follower execution failed for {}: {err}", input.instance);
-                                    input.instance
+                            continue;
-                                );
+                        }
-                                continue;
+                    };
-                            }
-                        };
 
                     if leader_diff == follower_diff {
-                        tracing::debug!("State diffs match between leader and follower.");
+                        log::debug!("State diffs match between leader and follower.");
                     } else {
-                        tracing::debug!("State diffs mismatch between leader and follower.");
+                        log::debug!("State diffs mismatch between leader and follower.");
                         Self::trace_diff_mode("Leader", &leader_diff);
                         Self::trace_diff_mode("Follower", &follower_diff);
                     }
 
                     if leader_receipt.logs() != follower_receipt.logs() {
-                        tracing::debug!("Log/event mismatch between leader and follower.");
+                        log::debug!("Log/event mismatch between leader and follower.");
-                        tracing::trace!("Leader logs: {:?}", leader_receipt.logs());
+                        log::trace!("Leader logs: {:?}", leader_receipt.logs());
-                        tracing::trace!("Follower logs: {:?}", follower_receipt.logs());
+                        log::trace!("Follower logs: {:?}", follower_receipt.logs());
                     }
 
                     if leader_receipt.status() != follower_receipt.status() {
-                        tracing::debug!(
+                        log::debug!(
                             "Mismatch in status: leader = {}, follower = {}",
                             leader_receipt.status(),
                             follower_receipt.status()

crates/core/src/main.rs

@@ -8,12 +8,10 @@ use revive_dt_core::{
     Geth, Kitchensink, Platform,
     driver::{Driver, State},
 };
-use revive_dt_format::{corpus::Corpus, metadata::MetadataFile};
+use revive_dt_format::{corpus::Corpus, metadata::Metadata};
 use revive_dt_node::pool::NodePool;
 use revive_dt_report::reporter::{Report, Span};
 use temp_dir::TempDir;
-use tracing::Level;
-use tracing_subscriber::{EnvFilter, FmtSubscriber, fmt::format::FmtSpan};
 
 static TEMP_DIR: LazyLock<TempDir> = LazyLock::new(|| TempDir::new().unwrap());
 
@@ -35,14 +33,7 @@ fn main() -> anyhow::Result<()> {
 }
 
 fn init_cli() -> anyhow::Result<Arguments> {
-    let subscriber = FmtSubscriber::builder()
+    env_logger::init();
-        .with_thread_ids(true)
-        .with_thread_names(true)
-        .with_env_filter(EnvFilter::from_default_env())
-        .with_span_events(FmtSpan::ENTER | FmtSpan::CLOSE)
-        .pretty()
-        .finish();
-    tracing::subscriber::set_global_default(subscriber)?;
 
     let mut args = Arguments::parse();
 
@@ -60,7 +51,7 @@ fn init_cli() -> anyhow::Result<Arguments> {
             args.temp_dir = Some(&TEMP_DIR);
         }
     }
-    tracing::info!("workdir: {}", args.directory().display());
+    log::info!("workdir: {}", args.directory().display());
 
     ThreadPoolBuilder::new()
         .num_threads(args.workers)
@@ -69,21 +60,21 @@ fn init_cli() -> anyhow::Result<Arguments> {
     Ok(args)
 }
 
-fn collect_corpora(args: &Arguments) -> anyhow::Result<HashMap<Corpus, Vec<MetadataFile>>> {
+fn collect_corpora(args: &Arguments) -> anyhow::Result<HashMap<Corpus, Vec<Metadata>>> {
     let mut corpora = HashMap::new();
 
     for path in &args.corpus {
         let corpus = Corpus::try_from_path(path)?;
-        tracing::info!("found corpus: {}", path.display());
+        log::info!("found corpus: {}", path.display());
         let tests = corpus.enumerate_tests();
-        tracing::info!("corpus '{}' contains {} tests", &corpus.name, tests.len());
+        log::info!("corpus '{}' contains {} tests", &corpus.name, tests.len());
         corpora.insert(corpus, tests);
     }
 
     Ok(corpora)
 }
 
-fn run_driver<L, F>(args: &Arguments, tests: &[MetadataFile], span: Span) -> anyhow::Result<()>
+fn run_driver<L, F>(args: &Arguments, tests: &[Metadata], span: Span) -> anyhow::Result<()>
 where
     L: Platform,
     F: Platform,
@@ -93,50 +84,34 @@ where
     let leader_nodes = NodePool::<L::Blockchain>::new(args)?;
     let follower_nodes = NodePool::<F::Blockchain>::new(args)?;
 
-    tests.par_iter().for_each(
+    tests.par_iter().for_each(|metadata| {
-        |MetadataFile {
+        let mut driver = Driver::<L, F>::new(
-             content: metadata,
+            metadata,
-             path: metadata_file_path,
+            args,
-         }| {
+            leader_nodes.round_robbin(),
-            // Starting a new tracing span for this metadata file. This allows our logs to be clear
+            follower_nodes.round_robbin(),
-            // about which metadata file the logs belong to. We can add other information into this
+        );
-            // as well to be able to associate the logs with the correct metadata file and case
-            // that's being executed.
-            let tracing_span = tracing::span!(
-                Level::INFO,
-                "Running driver",
-                metadata_file_path = metadata_file_path.display().to_string(),
-            );
-            let _guard = tracing_span.enter();
 
-            let mut driver = Driver::<L, F>::new(
+        match driver.execute(span) {
-                metadata,
+            Ok(_) => {
-                args,
+                log::info!(
-                leader_nodes.round_robbin(),
+                    "metadata {} success",
-                follower_nodes.round_robbin(),
+                    metadata.directory().as_ref().unwrap().display()
-            );
+                );
-
-            match driver.execute(span) {
-                Ok(_) => {
-                    tracing::info!(
-                        "metadata {} success",
-                        metadata.directory().as_ref().unwrap().display()
-                    );
-                }
-                Err(error) => {
-                    tracing::warn!(
-                        "metadata {} failure: {error:?}",
-                        metadata.file_path.as_ref().unwrap().display()
-                    );
-                }
             }
-        },
+            Err(error) => {
-    );
+                log::warn!(
+                    "metadata {} failure: {error:?}",
+                    metadata.file_path.as_ref().unwrap().display()
+                );
+            }
+        }
+    });
 
     Ok(())
 }
 
-fn execute_corpus(args: &Arguments, tests: &[MetadataFile], span: Span) -> anyhow::Result<()> {
+fn execute_corpus(args: &Arguments, tests: &[Metadata], span: Span) -> anyhow::Result<()> {
     match (&args.leader, &args.follower) {
         (TestingPlatform::Geth, TestingPlatform::Kitchensink) => {
             run_driver::<Geth, Kitchensink>(args, tests, span)?
@@ -150,12 +125,7 @@ fn execute_corpus(args: &Arguments, tests: &[MetadataFile], span: Span) -> anyho
     Ok(())
 }
 
-fn compile_corpus(
+fn compile_corpus(config: &Arguments, tests: &[Metadata], platform: &TestingPlatform, span: Span) {
-    config: &Arguments,
-    tests: &[MetadataFile],
-    platform: &TestingPlatform,
-    span: Span,
-) {
     tests.par_iter().for_each(|metadata| {
         for mode in &metadata.solc_modes() {
             match platform {

crates/format/Cargo.toml

@@ -13,7 +13,7 @@ alloy = { workspace = true }
 alloy-primitives = { workspace = true }
 alloy-sol-types = { workspace = true }
 anyhow = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
 semver = { workspace = true }
-serde = { workspace = true, features = ["derive"] }
+serde = { workspace = true, features = [ "derive" ] }
 serde_json = { workspace = true }

crates/format/src/corpus.rs

@@ -5,7 +5,7 @@ use std::{
 
 use serde::{Deserialize, Serialize};
 
-use crate::metadata::MetadataFile;
+use crate::metadata::Metadata;
 
 #[derive(Clone, Debug, Default, Serialize, Deserialize, Eq, PartialEq, Hash)]
 pub struct Corpus {
@@ -21,7 +21,7 @@ impl Corpus {
     }
 
     /// Scan the corpus base directory and return all tests found.
-    pub fn enumerate_tests(&self) -> Vec<MetadataFile> {
+    pub fn enumerate_tests(&self) -> Vec<Metadata> {
         let mut tests = Vec::new();
         collect_metadata(&self.path, &mut tests);
         tests
@@ -34,11 +34,11 @@ impl Corpus {
 /// Found tests are inserted into `tests`.
 ///
 /// `path` is expected to be a directory.
-pub fn collect_metadata(path: &Path, tests: &mut Vec<MetadataFile>) {
+pub fn collect_metadata(path: &Path, tests: &mut Vec<Metadata>) {
     let dir_entry = match std::fs::read_dir(path) {
         Ok(dir_entry) => dir_entry,
         Err(error) => {
-            tracing::error!("failed to read dir '{}': {error}", path.display());
+            log::error!("failed to read dir '{}': {error}", path.display());
             return;
         }
     };
@@ -47,7 +47,7 @@ pub fn collect_metadata(path: &Path, tests: &mut Vec<MetadataFile>) {
         let entry = match entry {
             Ok(entry) => entry,
             Err(error) => {
-                tracing::error!("error reading dir entry: {error}");
+                log::error!("error reading dir entry: {error}");
                 continue;
             }
         };
@@ -59,7 +59,7 @@ pub fn collect_metadata(path: &Path, tests: &mut Vec<MetadataFile>) {
         }
 
         if path.is_file() {
-            if let Some(metadata) = MetadataFile::try_from_file(&path) {
+            if let Some(metadata) = Metadata::try_from_file(&path) {
                 tests.push(metadata)
             }
         }

crates/format/src/input.rs

@@ -126,7 +126,7 @@ impl Input {
             .get(&self.instance)
             .ok_or_else(|| anyhow::anyhow!("ABI for instance '{}' not found", &self.instance))?;
 
-        tracing::trace!("ABI found for instance: {}", &self.instance);
+        log::trace!("ABI found for instance: {}", &self.instance);
 
         // Find function by selector
         let function = abi
@@ -140,7 +140,7 @@ impl Input {
                 )
             })?;
 
-        tracing::trace!("Functions found for instance: {}", &self.instance);
+        log::trace!("Functions found for instance: {}", &self.instance);
 
         let calldata_args = match &self.calldata {
             Some(Calldata::Compound(args)) => args,
@@ -155,7 +155,7 @@ impl Input {
             );
         }
 
-        tracing::trace!(
+        log::trace!(
             "Starting encoding ABI's parameters for instance: {}",
             &self.instance
         );

crates/format/src/metadata.rs

@@ -1,7 +1,6 @@
 use std::{
     collections::BTreeMap,
     fs::{File, read_to_string},
-    ops::Deref,
     path::{Path, PathBuf},
 };
 
@@ -16,29 +15,6 @@ pub const METADATA_FILE_EXTENSION: &str = "json";
 pub const SOLIDITY_CASE_FILE_EXTENSION: &str = "sol";
 pub const SOLIDITY_CASE_COMMENT_MARKER: &str = "//!";
 
-#[derive(Debug, Default, Deserialize, Clone, Eq, PartialEq)]
-pub struct MetadataFile {
-    pub path: PathBuf,
-    pub content: Metadata,
-}
-
-impl MetadataFile {
-    pub fn try_from_file(path: &Path) -> Option<Self> {
-        Metadata::try_from_file(path).map(|metadata| Self {
-            path: path.to_owned(),
-            content: metadata,
-        })
-    }
-}
-
-impl Deref for MetadataFile {
-    type Target = Metadata;
-
-    fn deref(&self) -> &Self::Target {
-        &self.content
-    }
-}
-
 #[derive(Debug, Default, Deserialize, Clone, Eq, PartialEq)]
 pub struct Metadata {
     pub cases: Vec<Case>,
@@ -59,7 +35,7 @@ impl Metadata {
             .filter_map(|mode| match mode {
                 Mode::Solidity(solc_mode) => Some(solc_mode),
                 Mode::Unknown(mode) => {
-                    tracing::debug!("compiler: ignoring unknown mode '{mode}'");
+                    log::debug!("compiler: ignoring unknown mode '{mode}'");
                     None
                 }
             })
@@ -114,7 +90,7 @@ impl Metadata {
         assert!(path.is_file(), "not a file: {}", path.display());
 
         let Some(file_extension) = path.extension() else {
-            tracing::debug!("skipping corpus file: {}", path.display());
+            log::debug!("skipping corpus file: {}", path.display());
             return None;
         };
 
@@ -126,14 +102,14 @@ impl Metadata {
             return Self::try_from_solidity(path);
         }
 
-        tracing::debug!("ignoring invalid corpus file: {}", path.display());
+        log::debug!("ignoring invalid corpus file: {}", path.display());
         None
     }
 
     fn try_from_json(path: &Path) -> Option<Self> {
         let file = File::open(path)
             .inspect_err(|error| {
-                tracing::error!(
+                log::error!(
                     "opening JSON test metadata file '{}' error: {error}",
                     path.display()
                 );
@@ -146,7 +122,7 @@ impl Metadata {
                 Some(metadata)
             }
             Err(error) => {
-                tracing::error!(
+                log::error!(
                     "parsing JSON test metadata file '{}' error: {error}",
                     path.display()
                 );
@@ -158,7 +134,7 @@ impl Metadata {
     fn try_from_solidity(path: &Path) -> Option<Self> {
         let spec = read_to_string(path)
             .inspect_err(|error| {
-                tracing::error!(
+                log::error!(
                     "opening JSON test metadata file '{}' error: {error}",
                     path.display()
                 );
@@ -187,7 +163,7 @@ impl Metadata {
                 Some(metadata)
             }
             Err(error) => {
-                tracing::error!(
+                log::error!(
                     "parsing Solidity test metadata file '{}' error: '{error}' from data: {spec}",
                     path.display()
                 );

crates/node-interaction/Cargo.toml

@@ -11,6 +11,6 @@ rust-version.workspace = true
 [dependencies]
 alloy = { workspace = true }
 anyhow = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
 once_cell = { workspace = true }
 tokio = { workspace = true }

crates/node-interaction/src/tokio_runtime.rs

@@ -51,13 +51,13 @@ impl TokioRuntime {
                 let nonce_task = spawn(interaction::<Nonce>(nonce_receiver));
 
                 if let Err(error) = transaction_task.await {
-                    tracing::error!("tokio transaction task failed: {error}");
+                    log::error!("tokio transaction task failed: {error}");
                 }
                 if let Err(error) = trace_task.await {
-                    tracing::error!("tokio trace transaction task failed: {error}");
+                    log::error!("tokio trace transaction task failed: {error}");
                 }
                 if let Err(error) = nonce_task.await {
-                    tracing::error!("tokio nonce task failed: {error}");
+                    log::error!("tokio nonce task failed: {error}");
                 }
             });
         });

crates/node/Cargo.toml

@@ -11,15 +11,15 @@ rust-version.workspace = true
 [dependencies]
 anyhow = { workspace = true }
 alloy = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
 
 revive-dt-node-interaction = { workspace = true }
 revive-dt-config = { workspace = true }
 
 serde_json = { workspace = true }
-
+ 
 sp-core = { workspace = true }
 sp-runtime = { workspace = true }
 
 [dev-dependencies]
-temp-dir = { workspace = true }
+temp-dir = { workspace = true }

crates/node/src/geth.rs

@@ -159,7 +159,7 @@ impl EthereumNode for Instance {
         let connection_string = self.connection_string();
         let wallet = self.wallet.clone();
 
-        tracing::debug!("Submitting transaction: {transaction:#?}");
+        log::debug!("Submitting transaction: {transaction:#?}");
 
         execute_transaction(Box::pin(async move {
             Ok(ProviderBuilder::new()

crates/node/src/kitchensink.rs

@@ -252,7 +252,7 @@ impl EthereumNode for KitchensinkNode {
         let url = self.rpc_url.clone();
         let wallet = self.wallet.clone();
 
-        tracing::debug!("Submitting transaction: {transaction:#?}");
+        log::debug!("Submitting transaction: {transaction:#?}");
 
         execute_transaction(Box::pin(async move {
             Ok(ProviderBuilder::new()

crates/node/src/pool.rs

@@ -62,7 +62,7 @@ where
 
 fn spawn_node<T: Node + Send>(args: &Arguments, genesis: String) -> anyhow::Result<T> {
     let mut node = T::new(args);
-    tracing::info!("starting node: {}", node.connection_string());
+    log::info!("starting node: {}", node.connection_string());
     node.spawn(genesis)?;
     Ok(node)
 }

crates/report/Cargo.toml

@@ -12,7 +12,8 @@ revive-dt-config = { workspace = true }
 revive-dt-format = { workspace = true }
 
 anyhow = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
 serde = { workspace = true }
 serde_json = { workspace = true }
 revive-solc-json-interface = { workspace = true }
+

crates/report/src/reporter.rs

@@ -192,7 +192,7 @@ impl Report {
         let file = File::create(&path).context(path.display().to_string())?;
         serde_json::to_writer_pretty(file, &self)?;
 
-        tracing::info!("report written to: {}", path.display());
+        log::info!("report written to: {}", path.display());
 
         Ok(())
     }

crates/solc-binaries/Cargo.toml

@@ -11,7 +11,7 @@ rust-version.workspace = true
 [dependencies]
 anyhow = { workspace = true }
 hex = { workspace = true }
-tracing = { workspace = true }
+log = { workspace = true }
 reqwest = { workspace = true }
 semver = { workspace = true }
 serde = { workspace = true }

crates/solc-binaries/src/cache.rs

@@ -25,7 +25,7 @@ pub(crate) fn get_or_download(
 
     let mut cache = SOLC_CACHER.lock().unwrap();
     if cache.contains(&target_file) {
-        tracing::debug!("using cached solc: {}", target_file.display());
+        log::debug!("using cached solc: {}", target_file.display());
         return Ok(target_file);
     }
 
@@ -37,10 +37,10 @@ pub(crate) fn get_or_download(
 }
 
 fn download_to_file(path: &Path, downloader: &GHDownloader) -> anyhow::Result<()> {
-    tracing::info!("caching file: {}", path.display());
+    log::info!("caching file: {}", path.display());
 
     let Ok(file) = File::create_new(path) else {
-        tracing::debug!("cache file already exists: {}", path.display());
+        log::debug!("cache file already exists: {}", path.display());
         return Ok(());
     };
 

crates/solc-binaries/src/download.rs

@@ -86,7 +86,7 @@ impl GHDownloader {
     /// Errors out if the download fails or the digest of the downloaded file
     /// mismatches the expected digest from the release [List].
     pub fn download(&self) -> anyhow::Result<Vec<u8>> {
-        tracing::info!("downloading solc: {self:?}");
+        log::info!("downloading solc: {self:?}");
         let expected_digest = List::download(self.list)?
             .builds
             .iter()