[webkit-reviews] review requested: [Bug 30809] buildbots should use --exit-after-N-failures : [Attachment 43773] upped limit to 40

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 24 08:16:32 PST 2009


Csaba Osztrogonac <ossy at webkit.org> has asked  for review:
Bug 30809: buildbots should use --exit-after-N-failures
https://bugs.webkit.org/show_bug.cgi?id=30809

Attachment 43773: upped limit to 40
https://bugs.webkit.org/attachment.cgi?id=43773&action=review

------- Additional Comments from Csaba Osztrogonac <ossy at webkit.org>
Yesterday master config was updated and it made QtBuildBot unhappy,
because we had 16 failing and 31 new tests. ( 31+16 > 20 ) On the
one hand failing tests counter shouldn't include new tests, that's why
I propose to modify it: https://bugs.webkit.org/show_bug.cgi?id=31829.
On the other hand if a patch committed make some layout tests fail,
and later an other patch make more failing tests, etc, we can easily 
lost which commit caused which fails. If we don't have this information,
debugging can be more difficult.

We make an effort to fix Qt related bugs ASAP, but quite a lot Qt-port 
maintainer sleep when american guys work. (because of timezone difference)

To make QtBuildbot and Qt guys happier, I suggest increasing 
the failure limit to 40.


More information about the webkit-reviews mailing list