Uploaded image for project: 'Apache Airflow'
  1. Apache Airflow
  2. AIRFLOW-5934

TRAINING ISSUE! Add extra CC: to the emails sent by Airflow

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.0.0
    • Fix Version/s: None
    • Component/s: breeze, core
    • Labels:

      Description

      This is a test issue that is used in Contributor's documentation to show an example issue for the first time contributors.

      Please do not merge the change even if you implement it!

      The example workflow that uses this JIRA issue is described in 

      https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#contribution-workflow-example

      We need to have a way to add extra CC: to  the emails sent by Airflow.

      1. You can see the information about how to configure email here email: https://airflow.readthedocs.io/en/latest/concepts.html#email-configuration
      2. Here is the class you should modify:  https://github.com/apache/airflow/blob/master/airflow/utils/email.py 
      3. Tests for email class are in: https://github.com/apache/airflow/blob/master/tests/utils/test_email.py 
      4. Run the tests in Local Virtualenv: https://github.com/apache/airflow/blob/master/LOCAL_VIRTUALENV.rst
      5. Run the tests in Breeze: https://github.com/apache/airflow/blob/master/BREEZE.rst
      6. Make a DRAFT PR as described in https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#contributions
      7. Run airflow webserver and scheduler and configure it to send emails and connect to a real SMTP server and test if the email is sent with the CC:'d account

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              potiuk Jarek Potiuk
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: