|Summary:||old-run-webkit-tests: save the list of tests actually run to a file|
|Product:||WebKit||Reporter:||Dirk Pranke <dpranke>|
|Component:||New Bugs||Assignee:||Dirk Pranke <dpranke>|
|Severity:||Normal||CC:||abarth, commit-queue, dbates, eric, ojan, ossy|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
Description Dirk Pranke 2011-04-15 14:03:11 PDT
old-run-webkit-tests: save the list of tests actually run to a file
Comment 2 Dirk Pranke 2011-04-15 14:08:16 PDT
I don't actually care that much whether we land this or not, I'm just posting it in case other people might find it useful.
Comment 3 Eric Seidel (no email) 2011-04-15 14:09:18 PDT
Comment on attachment 89849 [details] Patch I think this could be quite useful, honestly. Does NRWT do similar?
Comment 4 Dirk Pranke 2011-04-15 15:08:28 PDT
(In reply to comment #3) > (From update of attachment 89849 [details]) > I think this could be quite useful, honestly. Does NRWT do similar? Yes, NRWT saves a list per-worker.
Comment 5 WebKit Commit Bot 2011-04-15 20:59:56 PDT
Comment on attachment 89849 [details] Patch Clearing flags on attachment: 89849 Committed r84076: <http://trac.webkit.org/changeset/84076>
Comment 6 WebKit Commit Bot 2011-04-15 21:00:00 PDT
All reviewed patches have been landed. Closing bug.
Comment 7 Csaba Osztrogonác 2011-04-20 06:01:56 PDT
I think it is a useful feature for debugging, but we can get similar list on stdout with -v option. I have two minor problems with it: - ~ 3Mb sized list file for every build is hard disk killer on buildbot masters It would be better to make this feature optional,or upload gzipped list.(~130Kb) - absolute path: Why do we use absolute paths in this list? I think relative paths to LayoutTests directory is more than enough. Reopen to discuss about my proposed changes.
Comment 8 Eric Seidel (no email) 2011-04-20 12:10:15 PDT
I like both of those suggestions. Just putting it behind a flag seems reasonable.
Comment 9 Ojan Vafai 2011-04-20 12:16:50 PDT
(In reply to comment #8) > I like both of those suggestions. Just putting it behind a flag seems reasonable. Noooo. We have way too many flags already. There's no reason to store the absolute path. I expect storing relative paths will be enough to make this file reasonably sized. If not that, writing out the zipped file is surely sufficient.
Comment 10 Dirk Pranke 2011-04-20 13:10:11 PDT
I have no objection to either flags or shorter paths. I wouldn't bother gzipping the list. Or, feel free to revert it, even. We've made it this long without it :)
Comment 11 Dirk Pranke 2011-05-31 18:35:02 PDT
Csaba, do you have an opinion on whether just relative paths but not a gzipped list would be sufficient?