gocho1 commented on a change in pull request #7674: add s3 authentication 
method informations
URL: https://github.com/apache/incubator-druid/pull/7674#discussion_r285448467
 
 

 ##########
 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:
   done. I also reported properties that were on one side but not on other 
(when relevant).

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to