tests: do not set transaction timeout in tests that do not need it

Most of our transactions tests were build based with the same code which
often were copy pasted. The test were setting the default transaction
timeout to 10 seconds as they might have been based on the test that
validated the transactions expiration. Updated the tests to make sure
that the timeout is set in the test that need it.

The timeout made some tests to fail intermittently.

Fixes: redpanda-data#18184

Signed-off-by: Michał Maślanka <michal@redpanda.com>

tests/rptest/transactions/transactions_test.py

@@ -179,7 +179,6 @@ def init_transactions_creates_eos_topics_test(self):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         producer.init_transactions()
@@ -198,7 +197,6 @@ def simple_test(self):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         consumer1 = ck.Consumer({
@@ -276,7 +274,6 @@ def rejoin_member_test(self):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         group_name = "test"
@@ -334,7 +331,6 @@ def change_static_member_test(self):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         group_name = "test"
@@ -951,7 +947,6 @@ def consumer_offsets_retention_test(self):
         producer_conf = {
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': 'streams',
-            'transaction.timeout.ms': 10000,
         }
         producer = ck.Producer(producer_conf)
         consumer_conf = {
@@ -1137,7 +1132,6 @@ def init_transactions_authz_test(self):
         producer_cfg = {
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         }
 
         user = self.USER_1
@@ -1166,7 +1160,6 @@ def simple_authz_test(self):
         producer_cfg = {
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         }
         consumer_cfg = {
             'bootstrap.servers': self.redpanda.brokers(),
@@ -1352,7 +1345,6 @@ def do_upgrade_with_tx(self, selector):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         producer.init_transactions()
@@ -1375,7 +1367,6 @@ def do_upgrade_with_tx(self, selector):
         producer = ck.Producer({
             'bootstrap.servers': self.redpanda.brokers(),
             'transactional.id': '0',
-            'transaction.timeout.ms': 10000,
         })
 
         producer.init_transactions()
@@ -1457,7 +1448,6 @@ def delivery_callback(err, msg):
             producer = ck.Producer({
                 'bootstrap.servers': self.redpanda.brokers(),
                 'transactional.id': self._tx_id(i),
-                'transaction.timeout.ms': 10000,
             })
             producer.init_transactions()
             producer.begin_transaction()

tests/rptest/transactions/tx_coordinator_migration_test.py

@@ -53,7 +53,6 @@ def delivery_callback(err, msg):
             producer = ck.Producer({
                 'bootstrap.servers': self.redpanda.brokers(),
                 'transactional.id': self._tx_id(i),
-                'transaction.timeout.ms': 10000,
             })
             producer.init_transactions()
             producer.begin_transaction()