(kafka) branch 4.0 updated: KAFKA-18441: Fix flaky KafkaAdminClientTest#testAdminClientApisAuthenticationFailure (#18735)

[schofielaj]

This is an automated email from the ASF dual-hosted git repository.

schofielaj pushed a commit to branch 4.0
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/4.0 by this push:
     new a2d88f881c8 KAFKA-18441: Fix flaky 
KafkaAdminClientTest#testAdminClientApisAuthenticationFailure (#18735)
a2d88f881c8 is described below

commit a2d88f881c85fd0944c80319a29a44195c0dc4df
Author: PoAn Yang <pay...@apache.org>
AuthorDate: Thu Jan 30 16:01:20 2025 +0800

    KAFKA-18441: Fix flaky 
KafkaAdminClientTest#testAdminClientApisAuthenticationFailure (#18735)
    
    Reviewers: Lianet Magrans <lmagr...@confluent.io>, Chia-Ping Tsai 
<chia7...@gmail.com>, Andrew Schofield <aschofi...@confluent.io>
---
 .../java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
 
b/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
index 4cd4f048358..d74ddc00fc3 100644
--- 
a/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
+++ 
b/clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java
@@ -1782,7 +1782,10 @@ public class KafkaAdminClientTest {
     public void testAdminClientApisAuthenticationFailure() {
         Cluster cluster = mockBootstrapCluster();
         try (final AdminClientUnitTestEnv env = new 
AdminClientUnitTestEnv(Time.SYSTEM, cluster,
-                newStrMap(AdminClientConfig.REQUEST_TIMEOUT_MS_CONFIG, 
"1000"))) {
+                newStrMap(AdminClientConfig.REQUEST_TIMEOUT_MS_CONFIG, "1000",
+                    // Default "metadata.recovery.strategy" is rebootstrap. If 
it meets "retry.backoff.ms" (default is 100L),
+                    // following assertion will fail. Set it to none to avoid 
authentication error cleanup.
+                    AdminClientConfig.METADATA_RECOVERY_STRATEGY_CONFIG, 
"none"))) {
             env.kafkaClient().setNodeApiVersions(NodeApiVersions.create());
             
env.kafkaClient().createPendingAuthenticationError(cluster.nodes().get(0),
                     TimeUnit.DAYS.toMillis(1));