Skip to content

Commit a9ee084

Browse files
committed
dt/audit: Add 'rpc::service_unavailable' to upgrade test allow list
This test works by performing a rolling upgrade and verifying audit log function at each stage. With RPC-based audit logging enabled, the audit client on some node can race against service startup on some other, recently upgraded node, resulting in transient RPC error logs on the source node. In these cases the audit log client will immediately retry to avoid disruption, but the error log itself will trigger BadLogLines. Since the test is working as intended and there is no expectation of synchronization between RPC client and service on separate nodes, let's just add the specific rpc error log to the allow list for this test. Fixes CORE-15036 Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
1 parent f83d5c6 commit a9ee084

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

tests/rptest/tests/audit_log_test.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3374,7 +3374,11 @@ def stop_cond(records):
33743374
return len(self.read_all_from_audit_log(filter_fn, stop_cond))
33753375

33763376
@skip_fips_mode
3377-
@cluster(num_nodes=5, log_allow_list=RESTART_LOG_ALLOW_LIST)
3377+
@cluster(
3378+
num_nodes=5,
3379+
log_allow_list=RESTART_LOG_ALLOW_LIST
3380+
+ [re.compile("kafka/data/rpc - .*rpc::errc::service_unavailable.*")],
3381+
)
33783382
def test_audit_log_upgrade_all_nodes(self):
33793383
"""
33803384
Test that audit logging works on all nodes during rolling upgrade

0 commit comments

Comments
 (0)