windsock: kafka benches only include shotover sys_metrics on benches …

…using shotover (#1381)
shotover · Dec 4, 2023 · c42cb9c · c42cb9c
1 parent 11590c4
commit c42cb9c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git 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);
  }
  }