Opened 4 years ago
Last modified 4 years ago
#3624 new defect
[Nine] Doc inconsistency re: c['status'] config parameter
Reported by: | diegomarcov | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | undecided |
Version: | 0.9.0 | Keywords: | docs |
Cc: |
Description
Hi guys, thank you for the Nine project, this is getting awesome!
I'm porting my configuration to Nine, and I stumbled upon some problems when trying to port my HttpPushStatus service.
While reading the docs, I found the following inconsistency:
In http://docs.buildbot.net/latest/full.html#when-workers-go-missing there's an example that shows:
from buildbot.plugins import status # if jsmith commits a change, mail for the build is sent to jsmith@example.org notifier = status.MailNotifier(fromaddr="buildbot@example.org", sendToInterestedUsers=True, lookup="example.org") c['status'].append(notifier)
Several other examples are found along that same page that use the c['status'] configuration parameter.
However, another section of the same doc clearly says:
The whole c['status'] configuration parameter is gone.
Further reading suggests that indeed the c['status'] parameter is gone, but I think it'd be good to address this inconsistencies. I haven't yet found a way to port my HttpPushStatus service to Nine, because even the examples found in the docs are causing troubles on my config.
Thanks!
Hi Diego. This is right there is a need to cleanup all the references for cstatus?
Note There is an HttpStatusPush? already ported
https://docs.buildbot.net/current/manual/cfg-reporters.html#httpstatuspush
https://github.com/buildbot/buildbot/blob/master/master/buildbot/reporters/http.py