From 95a59d0cfb5f335d79f325bdd95ca5c9e3225dd7 Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Wed, 5 Nov 2025 12:29:41 +0100 Subject: [PATCH] DRA scheduler_perf: write JSON where perf-dash expects it BenchmarkPerfSchedulingExperimental_benchmark_dra_experimental_2025-11-05T07:10:33Z.json is ignored by perf-dash. It probably needs to be called BenchmarkPerfScheduling_benchmark_*. --- test/integration/scheduler_perf/scheduler_perf.go | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/test/integration/scheduler_perf/scheduler_perf.go b/test/integration/scheduler_perf/scheduler_perf.go index ba3e2143090..9174e75f1fe 100644 --- a/test/integration/scheduler_perf/scheduler_perf.go +++ b/test/integration/scheduler_perf/scheduler_perf.go @@ -1332,7 +1332,15 @@ func RunBenchmarkPerfScheduling(b *testing.B, configFile string, topicName strin } }) } - if err := dataItems2JSONFile(dataItems, strings.ReplaceAll(b.Name(), "/", "_")+"_benchmark_"+topicName); err != nil { + // Different top-level BenchmarkPerfScheduling* tests are supported as long as they use unique topic names. + // The final JSON file then is always called BenchmarkPerfScheduling_benchmark__.json + // because that is what perf-dash is configured to read: + // https://github.com/kubernetes/perf-tests/blob/581139e45e79cf04b9c2777b82677957f1e7f90b/perfdash/config.go#L520-L525 + namePrefix := b.Name() + namePrefix = regexp.MustCompile(`^BenchmarkPerfScheduling[^/]*`).ReplaceAllString(namePrefix, "BenchmarkPerfScheduling") + namePrefix = strings.ReplaceAll(namePrefix, "/", "_") + namePrefix += "_benchmark_" + topicName + if err := dataItems2JSONFile(dataItems, namePrefix); err != nil { b.Fatalf("unable to write measured data %+v: %v", dataItems, err) } }