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 2022/01/18 14:23:09 UTC

[GitHub] [airflow] Jorricks opened a new pull request #20923: Introduce notification_sent to SlaMiss view

Jorricks opened a new pull request #20923:
URL: https://github.com/apache/airflow/pull/20923


   Currently there is no way to view wether an SLA made it to the end user or not.
   We don't use email in our set-up as we use Slack/Mattermost notifications.
   This PR addresses that by simply adding notification_sent to the UI.


-- 
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] github-actions[bot] commented on pull request #20923: Introduce notification_sent to SlaMiss view

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #20923:
URL: https://github.com/apache/airflow/pull/20923#issuecomment-1015814735


   The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.


-- 
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] BasPH merged pull request #20923: Introduce notification_sent to SlaMiss view

Posted by GitBox <gi...@apache.org>.
BasPH merged pull request #20923:
URL: https://github.com/apache/airflow/pull/20923


   


-- 
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