From cce539495d38536807cac57054209bb0e9a2f1c8 Mon Sep 17 00:00:00 2001 From: Kashif Faraz Date: Sat, 23 Dec 2023 12:11:12 +0530 Subject: [PATCH] [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 4d2d308b2425..e4bc11b7ce44 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