refactor(walltime): rename PerfMetadata to WalltimeMetadata

Output filename also changes from perf.metadata to walltime.metadata.
This is a wire-breaking change that requires a coordinated server
rollout to read the new filename.

Author: not-matthias

crates/runner-shared/src/metadata.rs

@@ -12,7 +12,7 @@ use crate::module_symbols::MappedProcessModuleSymbols;
 use crate::unwind_data::MappedProcessUnwindData;
 
 #[derive(Serialize, Deserialize, Default)]
-pub struct PerfMetadata {
+pub struct WalltimeMetadata {
     /// The version of this metadata format.
     pub version: u64,
 
@@ -71,13 +71,13 @@ pub struct PerfMetadata {
     pub debug_info_by_pid: HashMap<pid_t, Vec<ModuleDebugInfo>>,
 }
 
-impl PerfMetadata {
+impl WalltimeMetadata {
     pub fn from_reader<R: std::io::Read>(reader: R) -> anyhow::Result<Self> {
-        serde_json::from_reader(reader).context("Could not parse perf metadata from JSON")
+        serde_json::from_reader(reader).context("Could not parse walltime metadata from JSON")
     }
 
     pub fn save_to<P: AsRef<Path>>(&self, path: P) -> anyhow::Result<()> {
-        let file = std::fs::File::create(path.as_ref().join("perf.metadata"))?;
+        let file = std::fs::File::create(path.as_ref().join("walltime.metadata"))?;
         const BUFFER_SIZE: usize = 256 * 1024 /* 256 KB */;
 
         let writer = BufWriter::with_capacity(BUFFER_SIZE, file);

src/executor/wall_time/profiler/mod.rs

@@ -6,6 +6,8 @@
 
 pub mod perf;
 
+const WALLTIME_METADATA_CURRENT_VERSION: u64 = 1;
+
 use crate::executor::ExecutorConfig;
 use crate::executor::ToolStatus;
 use crate::executor::helpers::command::CommandBuilder;

src/executor/wall_time/profiler/perf/mod.rs

@@ -12,6 +12,7 @@ use crate::executor::helpers::run_with_sudo::run_with_sudo;
 use crate::executor::helpers::run_with_sudo::wrap_with_sudo;
 use crate::executor::shared::fifo::FifoBenchmarkData;
 use crate::executor::shared::fifo::RunnerFifo;
+use crate::executor::wall_time::profiler::WALLTIME_METADATA_CURRENT_VERSION;
 use crate::executor::wall_time::profiler::perf::perf_executable::get_working_perf_executable;
 use crate::prelude::*;
 use anyhow::Context;
@@ -23,7 +24,7 @@ use runner_shared::artifacts::ArtifactExt;
 use runner_shared::artifacts::ExecutionTimestamps;
 use runner_shared::fifo::Command as FifoCommand;
 use runner_shared::fifo::IntegrationMode;
-use runner_shared::metadata::PerfMetadata;
+use runner_shared::metadata::WalltimeMetadata;
 use std::path::Path;
 use std::path::PathBuf;
 use std::{cell::OnceCell, process::ExitStatus};
@@ -36,7 +37,6 @@ pub(crate) mod setup;
 pub mod fifo;
 pub mod perf_executable;
 
-const PERF_METADATA_CURRENT_VERSION: u64 = 1;
 const PERF_PIPEDATA_FILE_NAME: &str = "perf.pipedata";
 
 pub struct PerfRunner {
@@ -328,8 +328,8 @@ impl BenchmarkData {
 
         debug!("Saving metadata");
         #[allow(deprecated)]
-        let metadata = PerfMetadata {
-            version: PERF_METADATA_CURRENT_VERSION,
+        let metadata = WalltimeMetadata {
+            version: WALLTIME_METADATA_CURRENT_VERSION,
             integration: self
                 .fifo_data
                 .integration