MINOR: Fix PerformanceService sys test (#18797)

This patch fixes the PerformanceService system test which was still using ZK.

```
================================================================================
SESSION REPORT (ALL TESTS)
ducktape version: 0.12.0
session_id:       2025-02-04--003
run time:         1 minute 42.629 seconds
tests run:        4
passed:           4                                                                                                                                                                         flaky:            0
failed:           0                                                                                                                                                                         ignored:          0
================================================================================
```

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
This commit is contained in:
David Jacot 2025-02-04 14:39:24 +01:00
parent 593c99b013
commit ff798dcdf1
1 changed files with 2 additions and 9 deletions

View File

@ -20,7 +20,6 @@ from ducktape.tests.test import Test
from kafkatest.services.kafka import KafkaService, quorum from kafkatest.services.kafka import KafkaService, quorum
from kafkatest.services.performance import ProducerPerformanceService, ConsumerPerformanceService, EndToEndLatencyService from kafkatest.services.performance import ProducerPerformanceService, ConsumerPerformanceService, EndToEndLatencyService
from kafkatest.services.performance import latency, compute_aggregate_throughput from kafkatest.services.performance import latency, compute_aggregate_throughput
from kafkatest.services.zookeeper import ZookeeperService
from kafkatest.version import DEV_BRANCH, LATEST_2_1, KafkaVersion from kafkatest.version import DEV_BRANCH, LATEST_2_1, KafkaVersion
@ -31,14 +30,8 @@ class PerformanceServiceTest(Test):
self.num_records = 10000 self.num_records = 10000
self.topic = "topic" self.topic = "topic"
self.zk = ZookeeperService(test_context, 1) if quorum.for_test(test_context) == quorum.zk else None
def setUp(self):
if self.zk:
self.zk.start()
@cluster(num_nodes=5) @cluster(num_nodes=5)
@matrix(version=[str(LATEST_2_1), str(DEV_BRANCH)], metadata_quorum=quorum.all) @matrix(version=[str(LATEST_2_1), str(DEV_BRANCH)], metadata_quorum=quorum.all_kraft)
def test_version(self, version=str(LATEST_2_1), metadata_quorum=quorum.zk): def test_version(self, version=str(LATEST_2_1), metadata_quorum=quorum.zk):
""" """
Sanity check out producer performance service - verify that we can run the service with a small Sanity check out producer performance service - verify that we can run the service with a small
@ -47,7 +40,7 @@ class PerformanceServiceTest(Test):
version = KafkaVersion(version) version = KafkaVersion(version)
self.kafka = KafkaService( self.kafka = KafkaService(
self.test_context, 1, self.test_context, 1,
self.zk, topics={self.topic: {'partitions': 1, 'replication-factor': 1}}, version=DEV_BRANCH) None, topics={self.topic: {'partitions': 1, 'replication-factor': 1}}, version=DEV_BRANCH)
self.kafka.start() self.kafka.start()
# check basic run of producer performance # check basic run of producer performance