Bug 29316 - commit-queue should give better feedback when failing a patch
Summary: commit-queue should give better feedback when failing a patch
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 28686
  Show dependency treegraph
 
Reported: 2009-09-16 15:19 PDT by Eric Seidel (no email)
Modified: 2009-09-25 10:41 PDT (History)
3 users (show)

See Also:


Attachments
First attempt at error message cleanups (10.28 KB, patch)
2009-09-16 15:26 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Fix args to not conflict with Exception.args (10.35 KB, patch)
2009-09-16 17:04 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Fix args to not conflict with Exception.args (10.37 KB, patch)
2009-09-16 17:43 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Limit error output length (10.94 KB, patch)
2009-09-24 14:52 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2009-09-16 15:19:25 PDT
commit-queue should give better feedback when failing a patch

I'm going to start by printing the logs from the failing command.  I will make further improvements soon, including giving information as to how to resolve the issue, or changing other parts of the commit-queue to not even bother failing a patch when it's not the user's fault (like when the ChangeLog is out of date, or maybe for a specific set of flakey tests).

This patch is about adding logging information when failing a patch.  I've been running the commit-queue with this code for a few hours and it seems to be doing fine. :)
Comment 1 Eric Seidel (no email) 2009-09-16 15:26:29 PDT
Created attachment 39663 [details]
First attempt at error message cleanups
Comment 2 Eric Seidel (no email) 2009-09-16 16:09:32 PDT
Example feedback:
https://bugs.webkit.org/show_bug.cgi?id=29280#c4

Obviously that message could be much less verbose.  That's the next step: write custom message parsers/translators for all the individual errors and tools.

In this example, we could parse out what layout test failed, and compose a more concise message that we post in the bug.
Comment 3 Eric Seidel (no email) 2009-09-16 17:04:42 PDT
Created attachment 39667 [details]
Fix args to not conflict with Exception.args
Comment 4 Eric Seidel (no email) 2009-09-16 17:43:05 PDT
Created attachment 39673 [details]
Fix args to not conflict with Exception.args
Comment 5 Adam Barth 2009-09-16 18:43:45 PDT
Comment on attachment 39673 [details]
Fix args to not conflict with Exception.args

ok
Comment 6 Eric Seidel (no email) 2009-09-16 19:32:07 PDT
Comment on attachment 39673 [details]
Fix args to not conflict with Exception.args

I rescind this patch.  I just noticed it's not logging to stdout as expected, I need to debug it further.
Comment 7 Eric Seidel (no email) 2009-09-24 14:52:31 PDT
Created attachment 40080 [details]
Limit error output length
Comment 8 Eric Seidel (no email) 2009-09-24 14:57:02 PDT
I've been testing this patch for the last week.  It seems to work just fine, and has dramatically improved usability of the queue now that people can see what fails.  I still need to deploy this type of reporting to a few other places in the code though.
Comment 9 Adam Barth 2009-09-25 10:02:17 PDT
Comment on attachment 40080 [details]
Limit error output length

That python array syntax is wild.
Comment 10 WebKit Commit Bot 2009-09-25 10:40:54 PDT
Comment on attachment 40080 [details]
Limit error output length

Clearing flags on attachment: 40080

Committed r48760: <http://trac.webkit.org/changeset/48760>
Comment 11 WebKit Commit Bot 2009-09-25 10:41:04 PDT
All reviewed patches have been landed.  Closing bug.