eladkal commented on code in PR #69877:
URL: https://github.com/apache/airflow/pull/69877#discussion_r3580742444
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -2394,6 +2394,19 @@ email:
type: string
example: ~
default: "airflow.utils.email.send_email_smtp"
+ email_notifier:
+ description: |
+ Dotted path to the notifier class used to deliver ``email_on_failure``
and
+ ``email_on_retry`` task alerts. The class must be a ``BaseNotifier``
subclass whose
+ constructor accepts ``to``, ``from_email``, ``subject`` and
``html_content`` keyword
+ arguments. Override it to route failure/retry alerts through a backend
other than SMTP
+ (for example an SES- or SendGrid-based notifier). Both the worker
task-runner path and
+ the DAG-processor callback path load the notifier from this single
option, so the
+ selected backend is used consistently regardless of how the task
failed.
+ version_added: 3.3.1
+ type: string
+ example: ~
+ default: "airflow.providers.smtp.notifications.smtp.SmtpNotifier"
Review Comment:
We want all email related settings out of core
https://github.com/apache/airflow/pull/30531
[[DISCUSS] Drop email integration from Airflow Core
](https://lists.apache.org/thread/tbnf2rcrowrp3ktltrhrcjvc53k58oxx) the
community disagreed with me about removing all of that as part of Airflow 3 and
now we are stuck with it :(
I think we should avoid adding more oil to this fire.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
