chore: ci logging management (#476)

Author: maksymsur

.config/nextest.toml

@@ -28,7 +28,7 @@ default-filter = 'not (binary(=integration_test_isolated) & test(/nightly_full_g
 
 [profile.ci.junit]
 path = "junit.xml"
-store-success-output = true
+store-success-output = false
 store-failure-output = true
 
 # Long timeout for tests that use production-sized FHE parameters (Default +
@@ -71,7 +71,7 @@ test-group = "k8s-integration"
 
 [profile.ci-nightly.junit]
 path = "junit.xml"
-store-success-output = true
+store-success-output = false
 store-failure-output = true
 
 # Long timeout + serialization for slow Default-param tests (same as ci profile).

Cargo.lock

@@ -19,6 +19,8 @@ members = [
     "observability",
     "tools/kms-health-check",
     "tools/generate-test-material",
+    "tools/kms-test-tracing",
+    "tools/kms-test-tracing-macro",
 ]
 
 # The backward-compatibility modules are excluded from the workspace to avoid dependency conflicts.
@@ -217,7 +219,7 @@ tracing = { version = "=0.1.44", features = ["log"] }  # Application instrumenta
 tracing-appender = "=0.2.3"  # Log file rotation - LOW RISK: tokio-rs team
 tracing-opentelemetry = "=0.32.1"  # OpenTelemetry integration - LOW RISK: tokio-rs team
 tracing-subscriber = { version = "=0.3.22", features = ["fmt", "std"] }  # Tracing subscriber - LOW RISK: tokio-rs team
-tracing-test = "=0.2.5"  # Tracing test utilities - LOW RISK: tokio-rs team
+kms-test-tracing = { path = "./tools/kms-test-tracing" }  # In-repo test harness (not crates.io `tracing-test`) - LOW RISK: in-repo
 trait-variant = "0.1.2"  # Trait variant generation - LOW RISK: rust-lang team utility
 typed-builder = "=0.21.0"  # Builder pattern macro - HIGH RISK: Individual maintainer (idanarye), despite 48M+ downloads
 url = { version = "=2.5.8", features = ["serde"] }  # URL parsing and manipulation - LOW RISK: servo team,

Cargo.toml

@@ -96,6 +96,7 @@ test-utils-cc = { path = "./test-utils-cc" }
 futures.workspace = true
 # Self-import for integration tests
 kms-core-client = { workspace = true, features = ["testing"] }
+kms-test-tracing.workspace = true
 serial_test.workspace = true
 test-context.workspace = true
 

core-client/Cargo.toml

@@ -53,7 +53,7 @@ use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::path::{Path, PathBuf};
 use std::str::FromStr;
-use std::sync::{Arc, Once};
+use std::sync::Arc;
 use std::time::Duration;
 use strum_macros::{Display, EnumString};
 use tfhe::FheTypes as TfheFheType;
@@ -1357,12 +1357,6 @@ pub async fn encrypt(
     ))
 }
 
-static INIT_LOG: Once = Once::new();
-
-pub fn init_testing() {
-    INIT_LOG.call_once(setup_logging);
-}
-
 pub fn setup_logging() {
     let file_appender = RollingFileAppender::new(Rotation::DAILY, "logs", "core-client.log");
     let file_and_stdout = file_appender.and(std::io::stdout);
@@ -1371,8 +1365,6 @@ pub fn setup_logging() {
     let log_level_str = std::env::var("RUST_LOG").unwrap_or_else(|_| "INFO".to_string());
     let log_level = tracing::Level::from_str(&log_level_str).unwrap_or(tracing::Level::INFO);
 
-    println!("Setting up logging with level: {log_level:?}");
-
     let subscriber = tracing_subscriber::fmt()
         .with_writer(file_and_stdout)
         .with_ansi(false)