Skip to content

Commit

Permalink
Merge pull request #24245 from dotnwat/CORE-8249
Browse files Browse the repository at this point in the history
[CORE-8249] Allow more time for segment clean up
  • Loading branch information
piyushredpanda authored Nov 22, 2024
2 parents 7792957 + 8e0c607 commit 514f1df
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions tests/rptest/tests/topic_delete_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,8 @@ def topic_storage_purged(redpanda, topic_name):
for topic_name, topic in ns.topics.items():
for p_id, p in topic.partitions.items():
for f in p.files:
redpanda.logger.info(f" {n.name}: {f}")
redpanda.logger.info(
f" {n.name}: {topic_name}_{p_id}_{f}")

return False

Expand Down Expand Up @@ -628,9 +629,13 @@ def topic_delete_unavailable_test(self, cloud_storage_type):
assert self.topic not in self.kafka_tools.list_topics()

# Local storage deletion should proceed even if remote can't
wait_until(lambda: topic_storage_purged(self.redpanda, self.topic),
timeout_sec=30,
backoff_sec=1)
wait_until(
lambda: topic_storage_purged(self.redpanda, self.topic),
timeout_sec=90,
backoff_sec=10,
err_msg=
"Local storage purge did not complete while cloud storage was unavailable"
)

# Erase timeout is hardcoded 60 seconds, wait long enough
# for it to give up.
Expand Down

0 comments on commit 514f1df

Please sign in to comment.