Skip to content

Commit a55fa35

Browse files
committed
tests/multi-server: reconnect test expects all 200 delivered, not 200/250
Over-provisioning num_messages above expected_messages masks real losses - we want the test to catch regressions in the reconnect path, not tolerate them. Drop the override so num_messages defaults to expected_messages (200) and the test fails if any message goes missing across the disconnect / reconnect cycle.
1 parent fd01627 commit a55fa35

1 file changed

Lines changed: 4 additions & 12 deletions

File tree

src/tests/multi-server/tests/kafka-produce-reconnect/short.ci.test.yml

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -30,18 +30,10 @@ loadgen:
3030
parallel: 1
3131
max_backlog: 1000
3232

33-
#
34-
# proto_load overshoots slightly - the `num_messages` cap stops
35-
# emission, but a handful of requests already in-flight in the
36-
# worker pool finish after that. Set num_messages well above
37-
# expected_messages so the consumer always has enough to count.
38-
#
39-
# expected_messages is what the consumer (kcat -c N) stops at,
40-
# so the kafka-consumer-summary line declares PASS as long as at
41-
# least `expected_messages` made it through the disconnect /
42-
# reconnect cycle.
43-
#
44-
loadgen_num_messages: 250
33+
# 30s x 10pps = 300, but num_messages caps at 200 so proto_load
34+
# stops at 200 emitted. Consumer expects exactly 200. If the
35+
# disconnect / reconnect path drops ANY message the test fails -
36+
# that's deliberate, we want to catch regressions, not hide them.
4537
expected_messages: 200
4638

4739
kafka_topic: fr-multi-server-reconnect-test

0 commit comments

Comments
 (0)