logs

Author: dylanlott

crates/sim/src/env.rs

@@ -9,7 +9,7 @@ use tokio::{
     select,
     sync::{mpsc, watch},
 };
-use tracing::{instrument, trace, trace_span};
+use tracing::{debug, error, instrument, trace, trace_span};
 use trevm::{
     db::{cow::CacheOnWrite, TryCachingDb},
     helpers::Ctx,
@@ -285,17 +285,15 @@ where
                     .unwrap_or_default();
                 let score = beneficiary_balance.saturating_sub(initial_beneficiary_balance);
 
-                trace!(
-                    gas_used = gas_used,
-                    score = %score,
+                debug!(
+                    gas_used,
+                    %score,
                     reverted = !success,
                     halted,
                     halt_reason = ?if halted { halt_reason } else { None },
                     revert_reason = if !success { reason } else { None },
                     "Simulation complete"
                 );
-
-                // Create the outcome
                 Ok(SimOutcomeWithCache { identifier, score, cache, gas_used })
             }
             Err(e) => Err(SignetEthBundleError::from(e.into_error())),
@@ -318,15 +316,18 @@ where
         // Run the bundle
         let trevm = match driver.run_bundle(trevm) {
             Ok(result) => result,
-            Err(e) => return Err(e.into_error()),
+            Err(e) => {
+                error!(?e, "run bundle returned an error");
+                return Err(e.into_error());
+            }
         };
 
         // Build the SimOutcome
         let score = driver.beneficiary_balance_increase();
         let gas_used = driver.total_gas_used();
         let cache = trevm.into_db().into_cache();
 
-        trace!(
+        debug!(
             gas_used = gas_used,
             score = %score,
             "Bundle simulation successful"
@@ -384,10 +385,10 @@ where
                                 let _ = c.blocking_send(candidate);
                                 return;
                             }
-                            trace!(gas_used = candidate.gas_used, max_gas, "Gas limit exceeded");
+                            debug!(gas_used = candidate.gas_used, max_gas, "Gas limit exceeded");
                         }
                         Err(e) => {
-                            trace!(?identifier, ?e, "Simulation failed");
+                            debug!(?identifier, ?e, "Simulation failed");
                         }
                     };
                     // fall through applies to all errors, occurs if