gocho1 commented on a change in pull request #7674: add s3 authentication method informations URL: https://github.com/apache/incubator-druid/pull/7674#discussion_r285013231
########## File path: docs/content/development/extensions-core/s3.md ########## @@ -65,6 +65,23 @@ As an example, to set the region to 'us-east-1' through system properties: `s3:GetObject` and `s3:PutObject` are basically required for pushing/loading segments to/from S3. If `druid.storage.disableAcl` is set to `false`, then `s3:GetBucketAcl` and `s3:PutObjectAcl` are additionally required to set ACL for objects. +### S3 authentication methods + +To connect to your S3 bucket (whether deep storage bucket or source bucket), Druid use the following credentials providers chain + +|order|type|details| +|--------|-----------|-------| +|1|Druid config file|Based on your runtime.properties if it contains values `druid.s3.accessKey` and `druid.s3.secretKey` | +|2|Custom properties file| Based on custom properties file where you can supply `sessionToken`, `accessKey` and `secretKey` values| Review comment: I'm wondering is that could not be better to remove informations in /content/configuration/index.md about s3 deep storage and just let the link to s3.md page. That would avoid information duplication, no? If OK, I'll just report `druid.storage.useS3aSchema` in s3.md and make the link ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
