(druid) branch master updated: Temporarily bump up the delay in auth IT from 5s to 10s. (#15765)

[za...@apache.org]

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

zachjsh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new a7918be268a Temporarily bump up the delay in auth IT from 5s to 10s. (#15765)
a7918be268a is described below

commit a7918be268adcef6f6c2bfa8b9bc6f82a4f3efb9
Author: Abhishek Radhakrishnan <ab...@gmail.com>
AuthorDate: Fri Jan 26 08:52:27 2024 -0800

    Temporarily bump up the delay in auth IT from 5s to 10s. (#15765)
    
    A more ideal/permanent fix would be to have status checks exposed by the services,
    but that'll require more code changes. So temporarily bump it to unblock CI now.
---
 .../apache/druid/tests/security/ITBasicAuthConfigurationTest.java  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/integration-tests/src/test/java/org/apache/druid/tests/security/ITBasicAuthConfigurationTest.java b/integration-tests/src/test/java/org/apache/druid/tests/security/ITBasicAuthConfigurationTest.java
index 04eb9da1f76..ea8e61ef317 100644
--- a/integration-tests/src/test/java/org/apache/druid/tests/security/ITBasicAuthConfigurationTest.java
+++ b/integration-tests/src/test/java/org/apache/druid/tests/security/ITBasicAuthConfigurationTest.java
@@ -82,11 +82,12 @@ public class ITBasicAuthConfigurationTest extends AbstractAuthConfigurationTest
   {
     super.setupHttpClientsAndUsers();
 
-    // Add a delay to allow propagation of credentials to all services
+    // Add a large enough delay to allow propagation of credentials to all services. It'd be ideal
+    // to have a "readiness" endpoint exposed by different services that'd return the version of auth creds cached.
     try {
-      Thread.sleep(5000);
+      Thread.sleep(10000);
     }
-    catch (Throwable t) {
+    catch (InterruptedException e) {
       // Ignore exception
     }
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org