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

stevel pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/branch-3.3 by this push:
     new b645e58  HADOOP-17433. Skipping network I/O in S3A getFileStatus(/) 
breaks ITestAssumeRole. (#2600)
b645e58 is described below

commit b645e58de295baf99651da1c31fc440bac651cbb
Author: Steve Loughran <[email protected]>
AuthorDate: Tue Jan 19 17:19:27 2021 +0000

    HADOOP-17433. Skipping network I/O in S3A getFileStatus(/) breaks 
ITestAssumeRole. (#2600)
    
    Contributed by Steve Loughran.
    
    Change-Id: Iece617be78e80fc7e956074eddf171f7763a2e66
---
 .../src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java
 
b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java
index 6b2290b..1c8dd77 100644
--- 
a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java
+++ 
b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestAssumeRole.java
@@ -394,8 +394,10 @@ public class ITestAssumeRole extends AbstractS3ATestBase {
         // when S3Guard is enabled, the restricted policy still
         // permits S3Guard record lookup, so getFileStatus calls
         // will work iff the record is in the database.
+        // probe the store using a path other than /, so a HEAD
+        // request is issued.
         forbidden("getFileStatus",
-            () -> fs.getFileStatus(ROOT));
+            () -> fs.getFileStatus(methodPath()));
       }
       forbidden("",
           () -> fs.listStatus(ROOT));


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to