This is an automated email from the ASF dual-hosted git repository.
kfaraz 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 cce539495d3 [Flaky test] Fix basic auth integration test (#15561)
cce539495d3 is described below
commit cce539495d38536807cac57054209bb0e9a2f1c8
Author: Kashif Faraz <[email protected]>
AuthorDate: Sat Dec 23 12:11:12 2023 +0530
[Flaky test] Fix basic auth integration test (#15561)
Database slowness while doing audits seems to be causing flakiness in auth
ITs.
The failing test is almost always
`ITBasicAuthConfigurationTest.test_avaticaQuery_datasourceAndContextParamsUser`
but in some rare cases, other tests fail too. Alternately, this failing
test has been seen to pass too.
It is most likely because the auth changes are not able to propagate in
time from
the coordinator to other services.
Fix: Just log the audits rather than persisting them to database.
Most audits have been newly added and it is okay to not have them persisted.
Moreover, logging audits can also be more beneficial while debugging an IT.
---
integration-tests/docker/environment-configs/common | 1 +
1 file changed, 1 insertion(+)
diff --git a/integration-tests/docker/environment-configs/common
b/integration-tests/docker/environment-configs/common
index 4d2d308b242..e4bc11b7ce4 100644
--- a/integration-tests/docker/environment-configs/common
+++ b/integration-tests/docker/environment-configs/common
@@ -77,6 +77,7 @@ druid_coordinator_kill_supervisor_period=PT10S
druid_coordinator_kill_supervisor_durationToRetain=PT0M
druid_coordinator_period_metadataStoreManagementPeriod=PT10S
druid_sql_planner_authorizeSystemTablesDirectly=true
+druid_audit_manager_type=log
# Testing the legacy config from https://github.com/apache/druid/pull/10267
# Can remove this when the flag is no longer needed
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
