Yeah, we can re-design status-bubble tooltip (https://webkit.org/b/201940) and include such information there.
Another idea to help with making style failures more noticeable is to have the Style failure displayed in-line on the review page along-with the reviewer's comments (https://webkit.org/b/202252). -Aakash > On Sep 19, 2019, at 1:41 PM, David Kilzer <ddkil...@webkit.org> wrote: > > I actually “missed” output from the stylebot insofar as I was expecting the > errors to be posted, and then I was going to reply to that automated message. > > I think it’s too easy to ignore style errors if you have to click to load > another website, though, so I filed this bug with some thoughts: > > Bug 201993: EWS style bot should show error output via semi-modal dialog > rather than requiring to click through > <https://bugs.webkit.org/show_bug.cgi?id=201993 > <https://bugs.webkit.org/show_bug.cgi?id=201993>> > > Maybe we could do something similar for layout test failures (a semi-modal > dialog that lists the errors, and links directly to the results rather than > having to click (twice) through the buildbot landing page? > > Dave > > > On Sep 19, 2019, at 9:45 AM, Aakash Jain <aakash_j...@apple.com > <mailto:aakash_j...@apple.com>> wrote: > >> On Jun 16, 2019, at 2:14 PM, Darin Adler <da...@apple.com >> <mailto:da...@apple.com>> wrote: >> >>> On Jun 15, 2019, at 9:13 PM, Aakash Jain <aakash_j...@apple.com >>> <mailto:aakash_j...@apple.com>> wrote: >>> >>>> 1) Do not upload archive (for layout-test-results) on bugzilla, instead >>>> upload it to another server, unzip it and post a link to the results.html. >>>> Pros: >>>> a) Engineers won't have to download the attachment, unzip it, look for >>>> failures, and then delete it from their disk. They can simply click the >>>> url to view the results. >>>> b) This approach will also reduce 2 comments per failure to 1 comment. >>>> Currently there are two comments per failure, one for failure details, >>>> second for bugzilla attachment. >>> >>> Great improvement to do this. >> >> We have implemented this in the new EWS. Layout test results are no longer >> added to EWS as attachments. Instead they are available to view in browser >> or download from the Buildbot build page. >> >>> The most confusing thing about build bot comments is all the “creation of >>> attachments” extra text with things like “attachment number” and “patch". >>> >>> However, it’s really nice that I can download a directory full of test >>> results easily. I’d like to see the EWS website still have that feature. >>> >>>> 4) When a patch becomes 'obsolete', tag the corresponding EWS comments as >>>> 'obsolete', so that they will be hidden. >>> >>> Incredibly valuable. >>> >>>> 5) Do not comment on bugzilla bug at all >>> >>> I think this makes sense. I don’t see a reason that test results need to be >>> comments. I think the “red bubble” in EWS already calls someone’s attention >>> to failures. >>> >>> If we want to augment it, we should think of what we are aiming at. I do >>> find it useful to see which tests are failing, and when I click on the red >>> bubble I don’t see that information. I have to click once to see the “log >>> of activities” then click on “results”, then see a confusing giant file >>> with lots of other information. At the bottom of that file the one thing I >>> want to know. >>> >>> A better hierarchy is to put that “what new tests are failing” summary >>> right t the top and let the logs be fallbacks, not the primary place to see >>> the features. >>> >>>> instead send email to the author of the patch. >>> >>> Why? I don’t think this should send any emails at all, unless the person >>> requested it. >>> >>>> Pros: less noisy, also this will allow to include more detailed >>>> information about the failure in email. >>> >>> I think the more detailed information should be on the webpage, not in an >>> email. >>> >>>> Cons: reviewers would have to click status-bubbles to see the failures, >>>> failure information is not immediately present in the comments. >>> >>> I think we should start with this approach, eliminating the comments >>> entirely. >> >> Following this suggestion, we eliminated the comments entirely in the new >> EWS. However, some people mentioned that since there are no comments, they >> do not get email notifications on failure, e.g.: https://webkit.org/b/200399 >> <https://webkit.org/b/200399>. Maybe we should have some kind of comments by >> EWS. Few ideas: >> >> 1) Comment on first failure for a patch, e.g.: "Some failures were noticed >> by EWS, please check the status bubbles". >> >> 2) Comment about success on all queues, e.g.: "Patch passed all EWS queues". >> >> >> What do you guys think? >> >>> >>> — Darin >> >
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev