[Buildbot] #3184: Rewrite MailNotifier to use Data API

Buildbot trac trac at buildbot.net
Sat Feb 7 20:25:29 UTC 2015


#3184: Rewrite MailNotifier to use Data API
-------------------+----------------------
Reporter:  dustin  |       Owner:
    Type:  task    |      Status:  new
Priority:  major   |   Milestone:  0.9.0b1
 Version:  master  |  Resolution:
Keywords:          |
-------------------+----------------------
Description changed by dustin:

Old description:

> Blocks #2684
>
> The MailNotifier still exists in `master`, but it's still mostly
> expecting the old Status hierarchy to exist, rather than the new Data
> API.
>
> It needs to be rewritten to listen for appropriate events from the Data
> API and send mail, with any additional information fetched through the
> Data API's "get" method.
>
> Note that the tests for !MailNotifier mock a lot of things out -
> incorrectly, now, so don't trust the tests!

New description:

 Blocks #2648

 The MailNotifier still exists in `master`, but it's still mostly expecting
 the old Status hierarchy to exist, rather than the new Data API.

 It needs to be rewritten to listen for appropriate events from the Data
 API and send mail, with any additional information fetched through the
 Data API's "get" method.

 Note that the tests for !MailNotifier mock a lot of things out -
 incorrectly, now, so don't trust the tests!

--

--
Ticket URL: <http://trac.buildbot.net/ticket/3184#comment:3>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the bugs mailing list