From 445f547675effefb9086c6487be2ec95f18638c3 Mon Sep 17 00:00:00 2001 From: Abhishek Balaji Radhakrishnan Date: Thu, 25 Jan 2024 22:31:42 -0800 Subject: [PATCH] Temporarily bump up the delay in auth IT from 5s to 10s. 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. --- .../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 04eb9da1f765..ea8e61ef3177 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 @@ protected void setupHttpClientsAndUsers() throws Exception { 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 } }