[ 
https://issues.apache.org/jira/browse/HADOOP-18542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17686734#comment-17686734
 ] 

ASF GitHub Bot commented on HADOOP-18542:
-----------------------------------------

steveloughran commented on PR #4262:
URL: https://github.com/apache/hadoop/pull/4262#issuecomment-1424712722

   the test which is a definite regression is
   ```
   ERROR] 
testConfigPropNotFound(org.apache.hadoop.fs.azurebfs.TestAccountConfiguration)  
Time elapsed: 0.19 s  <<< FAILURE!
   java.lang.AssertionError: Expected a 
org.apache.hadoop.fs.azurebfs.contracts.exceptions.TokenAccessProviderException 
to be thrown, but got the result: : 
"org.apache.hadoop.fs.azurebfs.oauth2.ClientCredsTokenProvider"
           at 
org.apache.hadoop.test.LambdaTestUtils.intercept(LambdaTestUtils.java:499)
           at 
org.apache.hadoop.test.LambdaTestUtils.intercept(LambdaTestUtils.java:384)
           at 
org.apache.hadoop.fs.azurebfs.TestAccountConfiguration.testMissingConfigKey(TestAccountConfiguration.java:399)
           at 
org.apache.hadoop.fs.azurebfs.TestAccountConfiguration.testConfigPropNotFound(TestAccountConfiguration.java:386)
           at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
           at 
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:566)
           at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
           at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
           at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
           at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
           at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
           at 
org.junit.runners.BlockJUnit4ClassRunner$1.evaluate(BlockJUnit4ClassRunner.java:100)
           at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:366)
           at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:103)
           at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:63)
           at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
   ```
   ...that is a test which came in with the mandatoryPassword code. you will 
have to edit CONFIG_KEYS  to remove the keys which are now optional.
   
   as to the others, I don't think they are related, more that you are testing 
through a non-HNS a/c and too much of the test code has drifted to being 
HNS-only.
   
   push up a patch for the CONFIG_KEYS change and I will download and test; if 
it works for me I'll be happy




> Azure Token provider requires tenant and client IDs despite being optional
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-18542
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18542
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs/azure, hadoop-thirdparty
>    Affects Versions: 3.3.2, 3.3.3, 3.3.4
>            Reporter: Carl
>            Priority: Major
>              Labels: pull-request-available
>
> The `AbfsConfiguration` class requires that we provide a tenant and client ID 
> when using the `MsiTokenProvider` class to fetch an authentication token. The 
> bug is that those fields are not required by the Azure API, which can infer 
> those fields when the call is made from an Azure instance.
> The fix is to make tenant and client ID optional when getting an Azure token 
> from the Azure Metadata Service.
> A fix has been submitted here: [https://github.com/apache/hadoop/pull/4262]
> The bug was introduced with HADOOP-17725  
> ([https://github.com/apache/hadoop/pull/3041/files])



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to