NEW 83078
garden-o-matic should show a *BIG* warning when a builder didn't run to completion
https://bugs.webkit.org/show_bug.cgi?id=83078
Summary garden-o-matic should show a *BIG* warning when a builder didn't run to compl...
Ojan Vafai
Reported 2012-04-03 14:55:01 PDT
For weeks, some bots have not run all the tests without anyone noticing. https://bugs.webkit.org/show_bug.cgi?id=83075 https://bugs.webkit.org/show_bug.cgi?id=83076 https://bugs.webkit.org/show_bug.cgi?id=83077 Increasingly, gardeners don't look at the waterfall since they just use garden-o-matic. As such they don't see when NRWT is bailing early due to too many unexpected crash/timeout. Also the waterfall doesn't give a big warning either (http://code.google.com/p/chromium/issues/detail?id=121229).
Attachments
Adam Barth
Comment 1 2012-04-03 15:11:03 PDT
Is there a bit in the results JSON we can look for? We already have UI for losing test coverage. We just need to trigger that UI when we bail early for too many failures.
Dirk Pranke
Comment 2 2012-04-03 15:13:49 PDT
(In reply to comment #1) > Is there a bit in the results JSON we can look for? We already have UI for losing test coverage. We just need to trigger that UI when we bail early for too many failures. No, not directly (I don't think), but it would certainly be easy to add one. I am tempted to want to make the bot (or at least the step) go purple when this happens, but normally purple is reserved for the infrastructure code failing, which isn't what's happening here.
Ojan Vafai
Comment 3 2012-04-03 15:26:35 PDT
Looks to me like there's an "interrupted" bit in full_results.json that covers this.
Ojan Vafai
Comment 4 2012-04-03 15:27:43 PDT
Dirk Pranke
Comment 5 2012-04-03 15:36:22 PDT
(In reply to comment #4) > Looks like it works: http://test-results.appspot.com/testfile?builder=Webkit%20Mac10.6&name=full_results.json has "interrupted":true. Yeah, I guess I had forgotten about that, but that's what that's for.
Eric Seidel (no email)
Comment 6 2012-04-03 16:23:43 PDT
I believe "interupted" is also used by results.html to show a warning. :)
Eric Seidel (no email)
Comment 7 2012-10-24 12:43:07 PDT
Still an issue?
Ojan Vafai
Comment 8 2012-10-24 14:11:51 PDT
AFAIK
Note You need to log in before you can comment on or make changes to this bug.