[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