fix(go): prevent running with valgrind

Author: not-matthias

…nner/wall_time/introspected_golang/go.sh …runner/helpers/introspected_golang/go.shsrc/run/runner/wall_time/introspected_golang/go.sh renamed to src/run/runner/helpers/introspected_golang/go.sh src/run/runner/wall_time/introspected_golang/go.sh renamed to src/run/runner/helpers/introspected_golang/go.sh

@@ -11,6 +11,13 @@ debug_log() {
 debug_log "Called with arguments: $*"
 debug_log "Number of arguments: $#"
 
+
+# Currently only walltime is supported
+if [ "${CODSPEED_RUNNER_MODE:-}" != "walltime" ]; then
+    echo "WARNING: Go benchmarks can only be run with walltime"
+    exit 1
+fi
+
 # Find the real go binary, so that we don't end up in infinite recursion
 REAL_GO=$(which -a go | grep -v "$(realpath "$0")" | head -1)
 if [ -z "$REAL_GO" ]; then

…ner/wall_time/introspected_golang/mod.rs …unner/helpers/introspected_golang/mod.rssrc/run/runner/wall_time/introspected_golang/mod.rs renamed to src/run/runner/helpers/introspected_golang/mod.rs src/run/runner/wall_time/introspected_golang/mod.rs renamed to src/run/runner/helpers/introspected_golang/mod.rs

@@ -1,5 +1,6 @@
 pub mod env;
 pub mod get_bench_command;
+pub mod introspected_golang;
 pub mod profile_folder;
 pub mod run_command_with_log_pipe;
 pub mod setup;

src/run/runner/helpers/mod.rs

@@ -2,6 +2,7 @@ use crate::prelude::*;
 use crate::run::runner::RunnerMode;
 use crate::run::runner::helpers::env::get_base_injected_env;
 use crate::run::runner::helpers::get_bench_command::get_bench_command;
+use crate::run::runner::helpers::introspected_golang;
 use crate::run::runner::helpers::run_command_with_log_pipe::run_command_with_log_pipe;
 use crate::run::runner::valgrind::helpers::ignored_objects_path::get_objects_path_to_ignore;
 use crate::run::runner::valgrind::helpers::introspected_nodejs::setup_introspected_nodejs;
@@ -90,11 +91,13 @@ pub async fn measure(
     .env(
         "PATH",
         format!(
-            "{}:{}",
+            "{}:{}:{}",
             setup_introspected_nodejs()
                 .map_err(|e| anyhow!("failed to setup NodeJS introspection. {}", e))?
-                .to_str()
-                .unwrap(),
+                .to_string_lossy(),
+            introspected_golang::setup_introspected_go()
+                .map_err(|e| anyhow!("failed to setup Go introspection. {}", e))?
+                .to_string_lossy(),
             env::var("PATH").unwrap_or_default(),
         ),
     );

src/run/runner/valgrind/measure.rs

@@ -5,8 +5,8 @@ use crate::run::instruments::mongo_tracer::MongoTracer;
 use crate::run::runner::executor::Executor;
 use crate::run::runner::helpers::env::{get_base_injected_env, is_codspeed_debug_enabled};
 use crate::run::runner::helpers::get_bench_command::get_bench_command;
+use crate::run::runner::helpers::introspected_golang::setup_introspected_go;
 use crate::run::runner::helpers::run_command_with_log_pipe::run_command_with_log_pipe;
-use crate::run::runner::wall_time::introspected_golang::setup_introspected_go;
 use crate::run::runner::{ExecutorName, RunData};
 use crate::run::{check_system::SystemInfo, config::Config};
 use async_trait::async_trait;

src/run/runner/wall_time/executor.rs

@@ -1,3 +1,2 @@
 pub mod executor;
-pub mod introspected_golang;
 pub mod perf;