chore: demote some logs from INFO to more appropriate TRACE/DEBUG/WARN

core/service/src/cryptography/signatures.rs

@@ -483,10 +483,10 @@ pub fn recover_address_from_ext_signature<S: SolStruct>(
     );
 
     let hash = data.eip712_signing_hash(domain);
-    tracing::info!("Public Data EIP-712 Message hash: {:?}", hash);
+    tracing::debug!("Public Data EIP-712 Message hash: {:?}", hash);
 
     let addr = sig.recover_address_from_prehash(&hash)?;
-    tracing::info!("Reconstructed address: {}", addr);
+    tracing::debug!("Reconstructed address: {}", addr);
 
     Ok(addr)
 }

core/service/src/engine/threshold/threshold_kms.rs

@@ -112,7 +112,7 @@ impl<
                 health_reporter
                     .set_not_serving::<CoreServiceEndpointServer<Self>>()
                     .await;
-                tracing::info!("Sat not serving");
+                tracing::trace!("Set not serving");
                 tracker.close();
                 tracker.wait().await;
                 mpc_abort_handle.abort();
@@ -211,7 +211,7 @@ impl<EP: Sync, UD: Sync, PD: Sync, KG: Sync, PP: Sync, CG: Sync, CM: Sync, BO: S
                 health_reporter
                     .set_not_serving::<CoreServiceEndpointServer<Self>>()
                     .await;
-                tracing::info!("Sat not serving");
+                tracing::trace!("Set not serving");
                 tracker.close();
                 tracker.wait().await;
                 mpc_abort_handle.abort();

core/service/src/vault/storage/crypto_material/utils.rs

@@ -200,7 +200,7 @@ pub fn log_data_exists<T: Display, U: Display, V: Display>(
             storage_info,
             pub_info
         ),
-        None => tracing::info!(
+        None => tracing::warn!(
             "{} with ID {} already exist, skipping generation",
             data_type,
             id

core/service/src/vault/storage/mod.rs

@@ -371,7 +371,7 @@ pub async fn delete_at_request_id<S: Storage>(
                 ))
             })
     } else {
-        tracing::info!(
+        tracing::warn!(
             "Tried to delete data with ID {} and type {}, but did not exist",
             request_id,
             data_type
@@ -403,7 +403,7 @@ pub async fn delete_at_request_and_epoch_id<S: StorageExt>(
                 ))
             })
     } else {
-        tracing::info!(
+        tracing::warn!(
             "Tried to delete data with ID {} and epoch {} and type {}, but did not exist",
             request_id,
             epoch_id,

observability/src/conf.rs

@@ -210,9 +210,11 @@ impl<'de> Deserialize<'de> for RetryConfig {
         }
 
         let helper = RetryConfigHelper::deserialize(deserializer)?;
-        println!(
+        tracing::info!(
             "Deserializing RetryConfig: max_retries={:?}, initial_delay_ms={:?}, max_delay_ms={:?}",
-            helper.max_retries, helper.initial_delay_ms, helper.max_delay_ms,
+            helper.max_retries,
+            helper.initial_delay_ms,
+            helper.max_delay_ms,
         );
 
         Ok(RetryConfig {
@@ -279,7 +281,7 @@ fn mode() -> ExecutionEnvironment {
     let res = env::var("RUN_MODE")
         .map(|enum_str| ExecutionEnvironment::from_str(enum_str.as_str()).unwrap_or_default())
         .unwrap_or_else(|_| ExecutionEnvironment::Local);
-    println!("RUN_MODE={res}");
+    tracing::info!("RUN_MODE={res}");
     res
 }
 
@@ -290,12 +292,12 @@ impl Settings<'_> {
     ///
     /// Returns an error if the configuration cannot be created or deserialized.
     pub fn init_conf<'de, T: Deserialize<'de> + std::fmt::Debug>(&self) -> Result<T, ConfigError> {
-        println!(
+        tracing::info!(
             "Initializing configuration with prefix: {}",
             self.env_prefix
         );
         if let Some(path) = self.path {
-            println!("Using config file: {path}");
+            tracing::info!("Using config file: {path}");
         }
 
         // NOTE: Environment variables that correspond to a sequence,