[Buildbot] #3481: Following doc for WebStatus in 0.8.12 leads to configuration that won't load... incomplete port of plugins from master?
Buildbot trac
trac at buildbot.net
Tue Mar 15 16:42:47 UTC 2016
#3481: Following doc for WebStatus in 0.8.12 leads to configuration that won't
load... incomplete port of plugins from master?
-------------------+--------------------
Reporter: dank | Owner:
Type: defect | Status: new
Priority: major | Milestone: 0.8.x
Version: 0.8.12 | Resolution:
Keywords: |
-------------------+--------------------
Changes (by dustin):
* type: undecided => defect
* milestone: undecided => 0.8.x
Old description:
> As described in
> https://lists.buildbot.net/pipermail/devel/2015-August/011904.html
>
> http://docs.buildbot.net/0.8.12/manual/cfg-
> statustargets.html#configuration claims this works:
> {{{
> from buildbot.plugins import status
> c['status'].append(status.WebStatus(8080))
> }}}
> But it fails with
> {{{
> exceptions.AttributeError: Plugin buildbot.status:WebStatus does not
> implement IStatusReceiver: An object has failed to implement
> interface <InterfaceClass buildbot...IStatusReceiver>
> }}}
> The workaround seems to be to ignore the docs and do
> {{{
> import buildbot.status.html
> c['status'].append(buildbot.status.html.WebStatus(http_port=8080))
> }}}
> Shouldn't either the doc or the code should be fixed so the documented
> sequence works?
New description:
As described in
https://lists.buildbot.net/pipermail/devel/2015-August/011904.html
http://docs.buildbot.net/0.8.12/manual/cfg-
statustargets.html#configuration claims this works:
{{{
from buildbot.plugins import status
c['status'].append(status.WebStatus(8080))
}}}
But it fails with
{{{
exceptions.AttributeError: Plugin buildbot.status:WebStatus does not
implement IStatusReceiver: An object has failed to implement
interface <InterfaceClass buildbot...IStatusReceiver>
}}}
The workaround seems to be to ignore the docs and do
{{{
import buildbot.status.html
c['status'].append(buildbot.status.html.WebStatus(http_port=8080))
}}}
Shouldn't either the doc or the code should be fixed so the documented
sequence works?
--
Comment:
I suspect that just adding an `implements(..)` would be sufficient.
--
Ticket URL: <http://trac.buildbot.net/ticket/3481#comment:3>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation
More information about the bugs
mailing list