[GitHub] [airflow] ashb commented on a change in pull request #6610: [AIRFLOW-6016] Remove redundant setting `remote_logging`

[GitBox <gi...@apache.org>]

ashb commented on a change in pull request #6610: [AIRFLOW-6016] Remove redundant setting `remote_logging`
URL: https://github.com/apache/airflow/pull/6610#discussion_r349701261
 
 

 ##########
 File path: docs/howto/write-logs.rst
 ##########
 @@ -157,9 +154,10 @@ First, to use the handler, ``airflow.cfg`` must be configured as follows:
     [core]
     # Airflow can store logs remotely in AWS S3, Google Cloud Storage or Elastic Search.
     # Users must supply an Airflow connection id that provides access to the storage
-    # location. If remote_logging is set to true, see UPDATING.md for additional
-    # configuration requirements.
-    remote_logging = True
+    # location. To set up correctly, see https://airflow.readthedocs.io/en/stable/howto/write-logs.html
 
 Review comment:
   General point: Don't link to readthedocs.io but to airflow.apache.org

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


With regards,
Apache Git Services