You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/23 16:08:49 UTC

[GitHub] [airflow] uranusjr opened a new issue #19781: A way to test email configuration

uranusjr opened a new issue #19781:
URL: https://github.com/apache/airflow/issues/19781


   ### Description
   
   Provide a way to test the email configurations work, perhaps in UI or CLI.
   
   ### Use case/motivation
   
   Airflow provides configurations for how emails should be sent. But after changing the configurations, it’s desirable to have a way to ensure they’re working by sending a test email, since otherwise it means the user is going to miss potentially critical email notifications and discover the error too late.
   
   The recent “Test Connection” addition on Connection entires is pretty awesome. Maybe we could do the same (although I’m not sure where the button should go). If there’s not a suitable UI location (or if the web server is not the correct process to send emails), we can add a CLI subcommand to do this instead.
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] subkanthi commented on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
subkanthi commented on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-1004202816


   What are the thoughts on providing the email template for the test email, would that also be defined in airflow.cfg?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] ashwin153 commented on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
ashwin153 commented on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-1001135395


   FWIW I have used `test@sink.sendgrid.net` to integration test email configurations in previous projects; `sink.sendgrid.net` is a "blackhole" domain that immediately deletes any emails that are sent to it. https://docs.sendgrid.com/ui/account-and-settings/safely-test-your-sending-speed


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-976782084


   Great idea. We have a new "email" type of connection. Sounds like we should simply handle it well. This conneciton is only used for user/password (and rest is kept in the config) but it could be a special case of "Test connection" button.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk edited a comment on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-976782084


   Great idea. We have a new "email" type of connection. Sounds like we should simply handle it in the same place. This conneciton is only used for user/password (and rest is kept in the config) but it could be a special case of "Test connection" button.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] eladkal commented on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
eladkal commented on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-977172443


   I really wonder if smtp configurations should be in airflow.cfg - is there a reason we keep it there other than this how earlier versions of Airflow did it ? :)
   The alternative is fully seperated email provider I guess?
   We used to have email provider but it wasnt a really Independent one so eventually it was removed and migrated back to core.
   
   In addition the smtp/emails are getting special treatment all over Airflow (see email_on_retry / email_on_failure)


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on issue #19781: A way to test email configuration

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #19781:
URL: https://github.com/apache/airflow/issues/19781#issuecomment-977298668


   Agree It could be just Hook and fully-fledged connection in Airflow core - does not have to be a provider.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org