airflow-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jakob Homan <jgho...@gmail.com>
Subject Re: Notifications
Date Wed, 05 Jul 2017 21:52:39 GMT
This is something we've discussed working on several times.  +1 for
the effort.

Thanks.

On 5 July 2017 at 13:07, Niels Zeilemaker <niels@zeilemaker.nl> 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" <
> daniel.imberman@gmail.com>:
>
> 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 <
> nielszeilemaker@godatadriven.com> 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
>>

Mime
View raw message