From c42cb9c8b92a1b580bae989eb639a1c4c4ddea5c Mon Sep 17 00:00:00 2001 From: Lucas Kent Date: Mon, 4 Dec 2023 20:38:46 +1100 Subject: [PATCH] windsock: kafka benches only include shotover sys_metrics on benches using shotover (#1381) --- shotover-proxy/benches/windsock/kafka.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/shotover-proxy/benches/windsock/kafka.rs b/shotover-proxy/benches/windsock/kafka.rs index 6ffca3f48..c93341c9c 100644 --- a/shotover-proxy/benches/windsock/kafka.rs +++ b/shotover-proxy/benches/windsock/kafka.rs @@ -232,16 +232,17 @@ impl Bench for KafkaBench { [("bencher".to_owned(), &bench_instance.instance)].into(); // only profile instances that we are actually using for this bench + if let Shotover::ForcedMessageParsed | Shotover::Standard = self.shotover { + profiler_instances.insert("shotover".to_owned(), &shotover_instance.instance); + } match self.topology { KafkaTopology::Single | KafkaTopology::Cluster1 => { profiler_instances.insert("kafka".to_owned(), &kafka_instance1.instance); - profiler_instances.insert("shotover".to_owned(), &shotover_instance.instance); } KafkaTopology::Cluster3 => { profiler_instances.insert("kafka1".to_owned(), &kafka_instance1.instance); profiler_instances.insert("kafka2".to_owned(), &kafka_instance2.instance); profiler_instances.insert("kafka3".to_owned(), &kafka_instance3.instance); - profiler_instances.insert("shotover".to_owned(), &shotover_instance.instance); } }