Improvement of SLA miss e-mail

[janpetrik2]

Hi,
I was wondering whether it would make sense to improve SLA email message, currently the e-mail looks like follow:

email_content = f"""\
            Here's a list of tasks that missed their SLAs:
            <pre><code>{task_list}\n<code></pre>
            Blocking tasks:
            <pre><code>{blocking_task_list}<code></pre>
            Airflow Webserver URL: {conf.get(section='webserver', key='base_url')}
            """

where task_list = "\n".join(sla.task_id + " on " + sla.execution_date.isoformat() for sla in slas). Source code for this part is here.

My idea for the improvement would be to include also link to task's log (i.e. ti.log_url) as it's done for failed DAG's alert. I think this change would make SLA miss email somehow more useful, although complete rewriting of SLAs mechanism might be a better approach.