Bug 83078 - garden-o-matic should show a *BIG* warning when a builder didn't run to completion
Summary: garden-o-matic should show a *BIG* warning when a builder didn't run to compl...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-03 14:55 PDT by Ojan Vafai
Modified: 2012-10-24 14:11 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ojan Vafai 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).
Comment 1 Adam Barth 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.
Comment 2 Dirk Pranke 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.
Comment 3 Ojan Vafai 2012-04-03 15:26:35 PDT
Looks to me like there's an "interrupted" bit in full_results.json that covers this.
Comment 4 Ojan Vafai 2012-04-03 15:27:43 PDT
Looks like it works: http://test-results.appspot.com/testfile?builder=Webkit%20Mac10.6&name=full_results.json has "interrupted":true.
Comment 5 Dirk Pranke 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.
Comment 6 Eric Seidel (no email) 2012-04-03 16:23:43 PDT
I believe "interupted" is also used by results.html to show a warning. :)
Comment 7 Eric Seidel (no email) 2012-10-24 12:43:07 PDT
Still an issue?
Comment 8 Ojan Vafai 2012-10-24 14:11:51 PDT
AFAIK