[ 
https://issues.apache.org/jira/browse/AIRFLOW-3572?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Victor Villas Bôas Chaves updated AIRFLOW-3572:
-----------------------------------------------
    Description: 
The docs on setting up remote logging with S3 currently states:

> To enable this feature, airflow.cfg must be configured as in this example:

... example ...

> In the above example, Airflow will try to use {{S3Hook('MyS3Conn')}}.

When in fact you can just leave `remote_log_conn_id =` (or nothing, because 
that's the default) if you have your credentials set in the process 
environment, because Airflow will fallback to boto3 config auto-detection.

This behavior is by design as the comment hints 
[here|https://github.com/apache/incubator-airflow/blob/master/airflow/contrib/hooks/aws_hook.py#L155-L158]
 so it should be documented.

  was:
The docs on setting up remote logging with S3 currently states:

> To enable this feature, airflow.cfg must be configured as in this example:

... example ...

> In the above example, Airflow will try to use {{S3Hook('MyS3Conn')}}.

When in fact you can just leave `remote_log_conn_id =` (or nothing, because 
that's the default) if you have your credentials set in the process 
environment, because Airflow will fallback to boto3 config auto-detection.


This behavior is by purpose as the comment hints 
[here|https://github.com/apache/incubator-airflow/blob/master/airflow/contrib/hooks/aws_hook.py#L155-L158]
 so it should be documented.


> Docs implies that providing remote_log_conn_id is mandatory
> -----------------------------------------------------------
>
>                 Key: AIRFLOW-3572
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-3572
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: Documentation
>    Affects Versions: 1.10.1
>            Reporter: Victor Villas Bôas Chaves
>            Priority: Minor
>
> The docs on setting up remote logging with S3 currently states:
> > To enable this feature, airflow.cfg must be configured as in this example:
> ... example ...
> > In the above example, Airflow will try to use {{S3Hook('MyS3Conn')}}.
> When in fact you can just leave `remote_log_conn_id =` (or nothing, because 
> that's the default) if you have your credentials set in the process 
> environment, because Airflow will fallback to boto3 config auto-detection.
> This behavior is by design as the comment hints 
> [here|https://github.com/apache/incubator-airflow/blob/master/airflow/contrib/hooks/aws_hook.py#L155-L158]
>  so it should be documented.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to