Skip to content

Commit

Permalink
tests/migrations: allow certain storage errors in logs
Browse files Browse the repository at this point in the history
Until Storage team can assess whether they are serious
  • Loading branch information
bashtanov committed Sep 12, 2024
1 parent 7153b46 commit 9a3ec5e
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions tests/rptest/tests/data_migrations_api_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@
from contextlib import nullcontext
import requests

MIGRATION_LOG_ALLOW_LIST = [
'Error during log recovery: cloud_storage::missing_partition_exception',
]


class TransferLeadersBackgroundThread:
def __init__(self, redpanda: RedpandaServiceBase, topic: str):
Expand Down Expand Up @@ -137,7 +141,7 @@ def migration_is_present(id: int):
f"Expected migration with id {migration_id} is not present")
return migration_id

@cluster(num_nodes=3)
@cluster(num_nodes=3, log_allow_list=MIGRATION_LOG_ALLOW_LIST)
def test_creating_and_listing_migrations(self):
self.finjector = Finjector(self.redpanda, self.test_context)

Expand Down Expand Up @@ -415,7 +419,7 @@ def cancel_inbound(self, migration_id, topic_name):
self.cancel(migration_id, topic_name)
self.assert_no_topics()

@cluster(num_nodes=4)
@cluster(num_nodes=4, log_allow_list=MIGRATION_LOG_ALLOW_LIST)
@matrix(use_alias=[True, False],
transfer_leadership=[True, False],
cancellation=[None] +
Expand Down

0 comments on commit 9a3ec5e

Please sign in to comment.