pgagnon commented on a change in pull request #5407: [AIRFLOW-4741] Include Sentry into core Airflow URL: https://github.com/apache/airflow/pull/5407#discussion_r295257844
########## File path: airflow/config_templates/default_airflow.cfg ########## @@ -200,6 +200,8 @@ worker_precheck = False # When discovering DAGs, ignore any files that don't contain the strings `DAG` and `airflow`. dag_discovery_safe_mode = True +# Sentry (https://docs.sentry.io) integration +sentry_dsn = Review comment: Are there more configuration parameters related to sentry that you might see being added in the future? If so it might be appropriate to move this setting to a new `[sentry]` section in the config. If that is the case it would be preferable to do this now before this is merged so that backward-compatibility concerns are minimized. ---------------------------------------------------------------- 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
