feat: add --show-full-output to bypass the new rolling buffer

Author: GuillaumeLagrange

src/cli/exec/mod.rs

@@ -88,6 +88,7 @@ fn build_orchestrator_config(
         skip_setup: args.shared.skip_setup,
         allow_empty: args.shared.allow_empty,
         go_runner_version: args.shared.go_runner_version,
+        show_full_output: args.shared.show_full_output,
         poll_results_options: PollResultsOptions::for_exec(),
     })
 }

src/cli/run/mod.rs

@@ -79,6 +79,7 @@ impl RunArgs {
                 skip_setup: false,
                 allow_empty: false,
                 go_runner_version: None,
+                show_full_output: false,
                 perf_run_args: PerfRunArgs {
                     enable_perf: false,
                     perf_unwinding_mode: None,
@@ -127,6 +128,7 @@ fn build_orchestrator_config(
         skip_setup: args.shared.skip_setup,
         allow_empty: args.shared.allow_empty,
         go_runner_version: args.shared.go_runner_version,
+        show_full_output: args.shared.show_full_output,
         poll_results_options,
     })
 }

src/cli/shared.rs

@@ -106,6 +106,10 @@ pub struct ExecAndRunSharedArgs {
     #[arg(long, env = "CODSPEED_GO_RUNNER_VERSION", value_parser = parse_version)]
     pub go_runner_version: Option<semver::Version>,
 
+    /// Show full executor output instead of a rolling buffer window
+    #[arg(long, default_value = "false")]
+    pub show_full_output: bool,
+
     #[command(flatten)]
     pub perf_run_args: PerfRunArgs,
 }

src/executor/config.rs

@@ -72,6 +72,8 @@ pub struct OrchestratorConfig {
     pub allow_empty: bool,
     /// The version of go-runner to install (if None, installs latest)
     pub go_runner_version: Option<Version>,
+    /// If true, show full executor output instead of a rolling buffer window
+    pub show_full_output: bool,
     /// Options controlling post-upload result polling and display
     pub poll_results_options: PollResultsOptions,
 }
@@ -201,6 +203,7 @@ impl OrchestratorConfig {
             skip_setup: false,
             allow_empty: false,
             go_runner_version: None,
+            show_full_output: false,
             poll_results_options: PollResultsOptions::for_exec(),
         }
     }

src/executor/orchestrator.rs

@@ -160,11 +160,15 @@ impl Orchestrator {
 
             let ctx = ExecutionContext::new(config, profile_folder);
 
-            activate_rolling_buffer(&part.label);
+            if !self.config.show_full_output {
+                activate_rolling_buffer(&part.label);
+            }
 
             run_executor(executor.as_ref(), self, &ctx, setup_cache_dir).await?;
 
-            deactivate_rolling_buffer();
+            if !self.config.show_full_output {
+                deactivate_rolling_buffer();
+            }
             all_completed_runs.push((ctx, executor.name()));
         }
 

src/project_config/merger.rs

@@ -160,6 +160,7 @@ mod tests {
             skip_setup: false,
             allow_empty: false,
             go_runner_version: None,
+            show_full_output: false,
             perf_run_args: PerfRunArgs {
                 enable_perf: true,
                 perf_unwinding_mode: None,
@@ -193,6 +194,7 @@ mod tests {
             skip_setup: false,
             allow_empty: false,
             go_runner_version: None,
+            show_full_output: false,
             perf_run_args: PerfRunArgs {
                 enable_perf: true,
                 perf_unwinding_mode: None,
@@ -228,6 +230,7 @@ mod tests {
             skip_setup: false,
             allow_empty: false,
             go_runner_version: None,
+            show_full_output: false,
             perf_run_args: PerfRunArgs {
                 enable_perf: false,
                 perf_unwinding_mode: None,