New issue
Advanced search Search tips

Issue 823934 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

luci-notify: add tryjob emails support

Project Member Reported by no...@chromium.org, Mar 20 2018

Issue description

Gatekepeer is used to send emails to owners of CLs for which a tryjob has failed.
GK works for LUCI based on Buildbot API emulation, which will eventually go away.

Foundation + Fuchsia infra teams have luci-notify.appspot.com service that can currently send notifications to
1) pre-configured email addresses when pre-configured builders fail/succeed/change latest status (used by Fuchsia)
2) email addresses specified in the build itself (used by CrOS)

from the Foundation team perspective, it would be natural to add tryjob support to luci-notify. It wouldn't depend on Buildbot API. The implementation would be fairly simple.
It is needed by v8 (sergiyb) and WebRTC (athom) teams.
 

Comment 1 by no...@chromium.org, Mar 20 2018

Sean, Tiffany, WDYT? Does your team wants to maintain the service that sends email notifications of failed builds?
Sounds like luci-notify was built for this purpose. Does it cover all of the users currently using gatekeeper?

Comment 3 by no...@chromium.org, Mar 21 2018

Cc: dpranke@chromium.org
Status: Available (was: Untriaged)
currently it does not cover all the users. Today we are using GK+API_Emulation combination for email notifications. This bug covers email notifications for failed tryjobs. I've just filed issue 823984 for CI build use case.
Wow, nice! I didn't know luci-notify is a thing already. I'd love to use it as is for V8, but https://luci-notify.appspot.com/ returns 404 for me. I've discovered [1], but it doesn't seem to have any fields for adding more projects. Also luci-notify is not listed at https://chrome-internal.googlesource.com/infra/infra_internal/+/master/doc/services.md (probably because it was not built by infra team in the first place).

[1]: https://chrome-internal.googlesource.com/infradata/config.git/+/master/configs/luci-notify/settings.cfg

Comment 5 by no...@chromium.org, Mar 28 2018

Sergiy, here is an example of project-level config:
https://fuchsia.googlesource.com/infra/config/+/master/services/luci-notify.cfg
HTTP 404 is WAI for now.
Thank you for the pointer. Looks like there is no filtering by step yet. That would be an essential feature too.

Sign in to add a comment