This is something we've discussed working on several times. +1 for the effort.
Thanks. On 5 July 2017 at 13:07, Niels Zeilemaker <[email protected]> wrote: > Hi Daniel, > > I've actually implemented an AirflowNotifier base class for others to > extend. This would allow for custom notifiers. > > Niels > > Op 5 jul. 2017 8:57 p.m. schreef "Daniel Imberman" < > [email protected]>: > > Hi Niels, > > Thank you for setting this up! I think this is a feature that would really > useful to a lot of people. Have you considered setting up a "notifier > plugin" so that people can create custom notifiers? The API seems pretty > consistent so I don't think it would be too much work to add. > > On Wed, Jul 5, 2017 at 11:40 AM Niels Zeilemaker < > [email protected]> wrote: > >> Hi All, >> >> I've opened a pull request ( >> https://github.com/apache/incubator-airflow/pull/2412) which introduces >> the concept of notifiers. >> I've made this change as I have a requirement to push status changes of >> failed/retried jobs to more than only email. Eg, I want to use slack in >> this case. >> Currently, we can only pass an email notifier, which I feel is pretty >> limited. >> >> My pull-request changes this and introduces Notifiers, which are triggered >> on status changes. I think it's almost there, but I have some problems >> trying to migrate the SLA stuff into the same concept. Let me know if you >> have any comments. Or if you feel this is the wrong approach to my > problem. >> >> Niels >>
