feluelle commented on a change in pull request #7461: [AIRFLOW-XXXX] Update
write-logs.rst, add instructions for logging to localstack S3
URL: https://github.com/apache/airflow/pull/7461#discussion_r381155588
##########
File path: docs/howto/write-logs.rst
##########
@@ -110,6 +110,11 @@ To enable this feature, ``airflow.cfg`` must be
configured as follows:
In the above example, Airflow will try to use ``S3Hook('MyS3Conn')``.
+If you are using LocalStack to emulate Amazon S3, you must create this entry
in airflow admin console:
+- Conn Id: MyS3Conn
+- Conn Type: S3
+- Extra: `{"host": "http://localstack:4572"}`
+
Review comment:
I am wondering if this should be an extra section or not. If it is really
only the host setting that needs to be added to get this working I would
suggest to keep it in the same section.
If not, we should explain the whole configuration from the beggining.
----------------------------------------------------------------
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