You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by dz...@apache.org on 2023/08/16 10:11:37 UTC

[drill] 02/02: Ensure that MiniDFS is using simple auth in BaseTestImpersonation.

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

dzamo pushed a commit to branch hadoop-impersonation-tests
in repository https://gitbox.apache.org/repos/asf/drill.git

commit 78bce12157111fdf731d2b50ed834e2622766f70
Author: James Turton <ja...@somecomputer.xyz>
AuthorDate: Wed Aug 16 12:11:02 2023 +0200

    Ensure that MiniDFS is using simple auth in BaseTestImpersonation.
---
 .../org/apache/drill/exec/impersonation/BaseTestImpersonation.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/exec/java-exec/src/test/java/org/apache/drill/exec/impersonation/BaseTestImpersonation.java b/exec/java-exec/src/test/java/org/apache/drill/exec/impersonation/BaseTestImpersonation.java
index b8c8b7a3b6..b928715f79 100644
--- a/exec/java-exec/src/test/java/org/apache/drill/exec/impersonation/BaseTestImpersonation.java
+++ b/exec/java-exec/src/test/java/org/apache/drill/exec/impersonation/BaseTestImpersonation.java
@@ -92,7 +92,6 @@ public class BaseTestImpersonation extends ClusterTest {
    */
   protected static void startMiniDfsCluster(String testClass, boolean isImpersonationEnabled) throws Exception {
     Preconditions.checkArgument(!Strings.isNullOrEmpty(testClass), "Expected a non-null and non-empty test class name");
-    UserGroupInformation.reset();
     dfsConf = new Configuration();
 
     // Set the MiniDfs base dir to be the temp directory of the test, so that all files created within the MiniDfs
@@ -104,6 +103,8 @@ public class BaseTestImpersonation extends ClusterTest {
       // Set the proxyuser settings so that the user who is running the Drillbits/MiniDfs can impersonate other users.
       dfsConf.set(String.format("hadoop.proxyuser.%s.hosts", processUser), "*");
       dfsConf.set(String.format("hadoop.proxyuser.%s.groups", processUser), "*");
+      dfsConf.set("hadoop.security.authentication", "simple");
+      UserGroupInformation.setConfiguration(dfsConf);
     }
 
     // Start the MiniDfs cluster