[webkit-dev] Requesting feedback about EWS comments on Bugzilla bugs

Michael Catanzaro mcatanzaro at igalia.com
Sun Jun 16 06:00:27 PDT 2019


1-4 all seem uncontroversial, especially with Tim's suggested 
improvement to immediately leave a "some queues failed" comment.

On Sat, Jun 15, 2019 at 11:13 PM, Aakash Jain <aakash_jain at apple.com> 
wrote:
> 5) Do not comment on bugzilla bug at all, instead send email to the 
> author of the patch.
> Pros: less noisy, also this will allow to include more detailed 
> information about the failure in email.
> Cons: reviewers would have to click status-bubbles to see the 
> failures, failure information is not immediately present in the 
> comments.

Well I think this would be OK, but next we'll be complaining about the 
emails. The underlying problem is excessive test flakiness. We're just 
not doing a very good job of tracking flaky tests and EWS isn't good at 
detecting flaky tests automatically. (Often a flaky test will fail 
twice in the same run, and that gets detected as a test failure.) 
Instead of reporting bugs for such tests, we're ignoring them because 
there are so many and we're so used to it.

commit-queue is able to report bugs when it detects a flaky patch, but 
EWS doesn't do this. That might be a positive change. Of course, EWS 
would need to be smart enough to not report a bug if the flakiness is 
triggered by the patch itself, which might not be simple to determine.

I don't have any concrete suggestions here. Just brainstorming.




More information about the webkit-dev mailing list