-
Notifications
You must be signed in to change notification settings - Fork 9k
HADOOP-19384. S3A: Add support for ProfileCredentialsProvider (#7284) #7690
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: branch-3.4
Are you sure you want to change the base?
HADOOP-19384. S3A: Add support for ProfileCredentialsProvider (#7284) #7690
Conversation
…#7284) Contributed by Venkatasubrahmanian Narayanan
🎊 +1 overall
This message was automatically generated. |
I am not seeing the stack trace you reported, FWIW. just ITestS3AContractOpen>AbstractContractOpenTest.testInputStreamReadNegativePosition, which is known about |
@steveloughran Then it might just be an issue with my local setup? There is nothing else from my end to report if the automated tests are all passing. |
put those stack traces up. failues in different configurations are always interesting -and finding them during development is better than finding them in production. do they go away when you use the normal credential providers? |
@steveloughran I did post the stacktraces in the JIRA earlier, I'd assumed you were responding after seeing those, guess it fell between the cracks. I'll have to change my setup to use the normal credentials providers, we can continue the conversation in the JIRA once I have those tests run. |
Description of PR
This commit adds a wrapper for the AWS
ProfileCredentialsProvider.
How was this patch tested?
The patch was tested by running the hadoop-aws integration tests with fs.s3a.aws.credentials.provider and fs.s3a.assumed.role.credentials.provider configured to only include org.apache.hadoop.fs.s3a.ProfileAWSCredentialsProvider. Buckets/endpoints used were in the us-east-1 region. 4 test failures that seem unrelated to this change have details in the JIRA.