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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
