From c7282472e97ad47e118670ba2e5a4b95b001a501 Mon Sep 17 00:00:00 2001 From: Ben Pope Date: Wed, 29 Sep 2021 14:00:41 +0100 Subject: [PATCH] ducktape: retention_policy - wait for controller PR #2428 changed the returned error code so that the error was retriable with a metadata refresh, but kafka-configs.sh doesn't retry. Call describe_topic to wait for the controller, as it performs a retry. Fix #2406 Signed-off-by: Ben Pope --- tests/rptest/tests/retention_policy_test.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/rptest/tests/retention_policy_test.py b/tests/rptest/tests/retention_policy_test.py index ee30f3579795..2f4fdbe46606 100644 --- a/tests/rptest/tests/retention_policy_test.py +++ b/tests/rptest/tests/retention_policy_test.py @@ -114,6 +114,10 @@ def test_changing_topic_retention_with_restart(self): self.redpanda.restart_nodes(self.redpanda.nodes) kafka_tools = KafkaCliTools(self.redpanda) + + # Wait for controller, alter configs doesn't have a retry loop + kafka_tools.describe_topic(self.topic) + # change retention bytes to preserve 15 segments kafka_tools.alter_topic_config( self.topic, {