1265871 - taskcluster-notify: handle IRC notifications

Status: RESOLVED FIXED

(Taskcluster :: General, defect)

Description

[(not currently active) Ted Mielczarek]

For things that don't report to Treeherder, it would be nice to be able to make them report status to IRC. Specifically, I currently have a manual task that builds Breakpad for Socorro to use:
https://github.com/mozilla/socorro/blob/master/scripts/breakpad-taskcluster.sh

It'd be great if I could have that report its status to irc.mozilla.org#breakpad. Socorro's travis builds already report status there, so it'd fit in pretty well.

Travis supports this in their yaml schema:
https://docs.travis-ci.com/user/notifications/#IRC-notification

Comment 1

[Dustin J. Mitchell [:dustin] (he/him)]

There are lots of cases for bots that will turn certain pulse messages into irc notifications -- be those pushes, task completions, treeherder requests, whatever.  So I think a much more general pulse-to-irc gateway service would be the right thing here, rather than some irc-specific code in the taskcluster worker.

I'm not sure that falls in the purview of TaskCluster, but we can keep it in TC:General until we figure out where it goes.

Comment 2

[Pete Moore [:pmoore][:pete]]

Agree a pulse->irc interface is preferable.

A job for https://github.com/taskcluster/taskcluster-hooks/ ?

Maybe with IRC settings in task.extra.notifications.irc ?

Not sure how well that would scale, we'd need to test.

Otherwise maybe a dedicated service (e.g deployed in heroku) listening against task-defined pulse exchange, looking at task.extra.notifications.irc, and then listening to applicable tasks in task-resolved/failed/exception exchanges based on what it picks up in task-defined?

What are your thoughts Jonas?

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

This seems like it'd be a perfect job for AWS lambda, if pulse fed to lambda.

Comment 4

[Jonas Finnemann Jensen (:jonasfj)]

I think this is a duplicate of bug 1088238, which is exactly what pmoore outlined.
I halfway started on it: https://github.com/taskcluster/taskcluster-notify
Pushed it yesterday, but haven't had time to finish it yet...

Comment 5

[Pete Moore [:pmoore][:pete]]

(In reply to Jonas Finnemann Jensen (:jonasfj) from comment #4)
> I think this is a duplicate of bug 1088238, which is exactly what pmoore
> outlined.
> I halfway started on it: https://github.com/taskcluster/taskcluster-notify
> Pushed it yesterday, but haven't had time to finish it yet...

Thanks Jonas!

Let's keep this bug open as the IRC part (and bug 1088238 can be about the email part). Probably taskcluster-notify should handle both.

Comment 6

[Dustin J. Mitchell [:dustin] (he/him)]

Pete, do we still need this open, now that tc-notify is done?

Comment 7

[Pete Moore [:pmoore][:pete]]

Thanks Dustin for following up! Indeed I think we can close.