[GitHub] [airflow] schnie commented on a change in pull request #5760: [AIRFLOW-5139] Allow custom ES configs

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

schnie commented on a change in pull request #5760: [AIRFLOW-5139] Allow custom ES configs
URL: https://github.com/apache/airflow/pull/5760#discussion_r312243997
 
 

 ##########
 File path: airflow/config_templates/default_airflow.cfg
 ##########
 @@ -591,6 +591,11 @@ json_format = False
 # Log fields to also attach to the json output, if enabled
 json_fields = asctime, filename, lineno, levelname, message
 
+[elasticsearch_configs]
+
+use_ssl = False
+verify_certs = False
 
 Review comment:
   @dimberman wondering if we should mirror the defaults that the es client uses here so we don't change any behavior. `verify_certs` is `True` by default here: https://github.com/elastic/elasticsearch-py/blob/master/elasticsearch/connection/http_requests.py#L47-L48

----------------------------------------------------------------
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