Skip to content

Commit 779adc5

Browse files
committed
rptest: use 10 nodes in OMB validation tests
Drop the assumed number of nodes from 12 to 10 in OMB validation tests as that's the default number of nodes in duck.py and also the number we want to standardize on for all HTT tests.
1 parent 049e2f2 commit 779adc5

File tree

1 file changed

+13
-8
lines changed

1 file changed

+13
-8
lines changed

tests/rptest/redpanda_cloud_tests/omb_validation_test.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@
3535

3636
class OMBValidationTest(RedpandaTest):
3737

38+
# The numbers of nodes we expect to run with - this value (10) is the default
39+
# for duck.py so these tests should just work with that default, but not necessarily
40+
# any less than that.
41+
CLUSTER_NODES = 10
42+
3843
# common workload details shared among most/all test methods
3944
WORKLOAD_DEFAULTS = {
4045
"topics": 1,
@@ -141,7 +146,7 @@ def _consumer_count(self, egress_rate) -> int:
141146
def _mb_to_mib(self, mb):
142147
return math.floor(0.9537 * mb)
143148

144-
@cluster(num_nodes=12)
149+
@cluster(num_nodes=CLUSTER_NODES)
145150
def test_max_connections(self):
146151
tier_limits = self.tier_limits
147152

@@ -150,7 +155,7 @@ def test_max_connections(self):
150155

151156
PRODUCER_TIMEOUT_MS = 5000
152157
OMB_WORKERS = 4
153-
SWARM_WORKERS = 7
158+
SWARM_WORKERS = 5
154159

155160
# OMB parameters
156161
#
@@ -285,7 +290,7 @@ def test_max_connections(self):
285290
finally:
286291
self.rpk.delete_topic(swarm_topic_name)
287292

288-
@cluster(num_nodes=12)
293+
@cluster(num_nodes=CLUSTER_NODES)
289294
def test_max_partitions(self):
290295
tier_limits = self.tier_limits
291296

@@ -320,14 +325,14 @@ def test_max_partitions(self):
320325
self.redpanda,
321326
"ACK_ALL_GROUP_LINGER_1MS_IDEM_MAX_IN_FLIGHT",
322327
(workload, validator),
323-
num_workers=10,
328+
num_workers=self.CLUSTER_NODES - 1,
324329
topology="ensemble")
325330
benchmark.start()
326331
benchmark_time_min = benchmark.benchmark_time() + 5
327332
benchmark.wait(timeout_sec=benchmark_time_min * 60)
328333
benchmark.check_succeed()
329334

330-
@cluster(num_nodes=12)
335+
@cluster(num_nodes=CLUSTER_NODES)
331336
def test_common_workload(self):
332337
tier_limits = self.tier_limits
333338

@@ -373,14 +378,14 @@ def test_common_workload(self):
373378
benchmark = OpenMessagingBenchmark(self._ctx,
374379
self.redpanda,
375380
driver, (workload, validator),
376-
num_workers=10,
381+
num_workers=self.CLUSTER_NODES - 1,
377382
topology="ensemble")
378383
benchmark.start()
379384
benchmark_time_min = benchmark.benchmark_time() + 5
380385
benchmark.wait(timeout_sec=benchmark_time_min * 60)
381386
benchmark.check_succeed()
382387

383-
@cluster(num_nodes=12)
388+
@cluster(num_nodes=CLUSTER_NODES)
384389
def test_retention(self):
385390
tier_limits = self.tier_limits
386391

@@ -439,7 +444,7 @@ def test_retention(self):
439444
benchmark = OpenMessagingBenchmark(self._ctx,
440445
self.redpanda,
441446
driver, (workload, validator),
442-
num_workers=10,
447+
num_workers=self.CLUSTER_NODES - 1,
443448
topology="ensemble")
444449
benchmark.start()
445450
benchmark_time_min = benchmark.benchmark_time() + 5

0 commit comments

Comments
 (0)