Re: [webkit-dev] Please be careful with webkitpy changes!

2015-04-02 Thread Maciej Stachowiak
You said it did not detect the failure until many builds later. That seems bad. People expect EWS validation to happen on their bug, not out of band 10-13 builds later. Is there any way to fix this limitation? That seems better than asking people to remember exceptions about patches that EWS

Re: [webkit-dev] Please be careful with webkitpy changes!

2015-04-02 Thread Osztrogonác Csaba
Hi, There are different issues here: - EWS bots don't run webkitpy tests, except the Mac WK1 and WK2. (running webkitpy tests is part of run tests) Maybe we should run webkitpy tests on the non tester EWS bots too. - poor webkitpy test coverage: Windows buildbots should have noticed the

Re: [webkit-dev] Please be careful with webkitpy changes!

2015-04-02 Thread Ryosuke Niwa
Right now, EWS restarts after every 20 iterations or so. We could expedite the process by doing it more frequently. However, I can't think of a way we can test a webkitpy patch that affects EWS on EWS since it's webkitpy code that's applying the patch and running the tests, etc… On Thu, Apr 2,

Re: [webkit-dev] Please be careful with webkitpy changes!

2015-04-02 Thread youenn fablet
Hi, Do you have any additional information on why the first patch failed? Can the problem be reproduced? Thanks Y Le 1 avr. 2015 21:43, Brent Fulgham bfulg...@apple.com a écrit : Hi Everyone, We lost Windows EWS coverage for the past 36 hours due to a very benign-appearing change to some