70544 2011-10-20 13:58:36 -0700 Tools should more eagerly contact people who may have broken the build 2011-11-08 19:45:29 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 36581 1 simon.fraser webkit-unassigned abarth dpranke eric jchaffraix simon.fraser tomz oldest_to_newest 487858 0 simon.fraser 2011-10-20 13:58:36 -0700 When you check in something that breaks the build (compile failure, test failure), we should more proactively contact people who are on the hook for the bustage. This should include: * sheriffbot pinging them on IRC (this used to happen) * adding comments to bugs * sending email * sending text message to phone numbers 487860 1 abarth 2011-10-20 14:00:15 -0700 We have all the code to do this, but it's current disabled because it was too spammy. Maybe restricting it to compile failures would be a good way to make it non-spammy? 487867 2 simon.fraser 2011-10-20 14:07:02 -0700 We have so much unnoticed build and test bustage that I'd rather it be too spammy. It should only spam those who've committed, which should make people more cautious before committing. That can only be a good thing. 487874 3 abarth 2011-10-20 14:12:36 -0700 The first step is (roughly) to revert http://trac.webkit.org/changeset/90636 487899 4 eric 2011-10-20 15:00:24 -0700 I support this. Our last (long running) attempt at this seemed a failure. I think what you're seeking is cultural change more than technical change. Perhaps the technical can help the cultural with the right leadership. 487911 5 abarth 2011-10-20 15:17:17 -0700 Some ideas for reducing spam: 1) Instead of using Pass Fail Fail as the signal to bug people, we can use Pass Pass Fail Fail. That will filter out noise from tests that fail 90% of the time. 2) Rate limit messages from each test. If we've bugged someone about a given test already in the past 24 hours, don't bug anyone this time. 489617 6 jchaffraix 2011-10-24 13:58:53 -0700 It's sad that the tree is more or less always half red / orange these days (and that's only the core builders). Currently if you want to make sure your patches don't break anything, you have first to get whoever broke the build to fix it first. Having sheriffbot yell at people had the upside of making it obvious that the person was contacted about it and when he/she heard about the breakage (in case a rollout needs to be done). 489624 7 abarth 2011-10-24 14:01:38 -0700 There are also too many builders in the core category. If a port hasn't even compiled in days, it should be removed. 498616 8 eric 2011-11-08 19:45:29 -0800 We've also discussed making the sheriff-bot more agressive about contacting people who break the build in the past. See bug 36581.