Bug 59008 – [buildbot]Show more recent builds on builder page

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Csaba Osztrogonác]

If I click on a builder on waterfall page, 
I can see only 5 builds under the recent builds.

It would be great if we can see more than 5 to make gardening more effective.
I know that I can add "?numbuilds=100" to the URL, but it isn't so user friendly.
Can't we add this option to master.cfg somehow?

[Peter Gal]

(In reply to comment #0)
> If I click on a builder on waterfall page, 
> I can see only 5 builds under the recent builds.
> 
> It would be great if we can see more than 5 to make gardening more effective.
> I know that I can add "?numbuilds=100" to the URL, but it isn't so user friendly.
> Can't we add this option to master.cfg somehow?

It is possible to pass 'numbuilds' argument to the WebStatus in the master.cfg, but that does not affects the builder's page. But we can override the build in template which will contain the required extra links.

[Peter Gal]

Created attachment 90362 [details]
template patch

Simply added these lines to the built-in template:

( Show: <a href="{{ builder_url }}">default</a>
{% for count in [25, 50, 100, 200] %}
<a href="{{ builder_url }}?numbuilds={{ count }}">{{ count }}</a>
{% endfor %})

This will show five links with different 'numbuilds' parameters.

[Csaba Osztrogonác]

Comment on attachment 90362 [details]
template patch

yay, shibbbyyyy! 

r=me

[Csaba Osztrogonác]

Adam, could you run a reconfigure on the buildmaster after it is landed?

[Peter Gal]

BTW, this template is from Buildbot v0.8.3, so it should apply cleanly to the apple's buildmaster.

[Adam Roben (:aroben)]

It would be slightly clearer to check in the unmodified template first, and then customize it after that in a separate change.

I can take care of updating the master after this lands.

[Csaba Osztrogonác]

Comment on attachment 90362 [details]
template patch

OK, let's check in the unmodified template first, and then patch it.

[Peter Gal]

Created attachment 90375 [details]
base template

This is the built-in template from buildbot v0.8.3.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 90362 [details]:

http/tests/xmlhttprequest/logout.html bug 52047 (author: [e-mail])
The commit-queue is continuing to process your patch.

[Csaba Osztrogonác]

Comment on attachment 90375 [details]
base template

Landed in http://trac.webkit.org/changeset/84421

[Csaba Osztrogonác]

Modifying the default template landed in http://trac.webkit.org/changeset/84423

[WebKit Review Bot]

http://trac.webkit.org/changeset/84423 might have broken Chromium Win Release

[Peter Gal]

(In reply to comment #11)
> Modifying the default template landed in http://trac.webkit.org/changeset/84423

Cool, Thanks!