WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
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
Looks like it works:
http://test-results.appspot.com/testfile?builder=Webkit%20Mac10.6&name=full_results.json
has "interrupted":true.
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.
Top of Page
Format For Printing
XML
Clone This Bug