<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1251"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">(re-sent from correct address)<div><br><div><div><div>11  2014 .,  3:59, Maciej Stachowiak &lt;<a href="mailto:mjs@apple.com">mjs@apple.com</a>&gt; ():</div><br class="Apple-interchange-newline"><blockquote type="cite">So it seems like the extra request for people using webkitbot rollout is to add diagnostic information to the rollout bug, and wait a reasonable period before cq+ing it. Is that something everyone could live with?</blockquote></div><div><br></div><div>In the past, I've seen people overlook comments in rollout bugs more frequently than in original bugs, so I usually added the diagnostic information to the original bug. I think that it's more relevant there, as that's where people will be continuing the work. Knowing what the failure symptoms were is certainly relevant when reviewing a new iteration of the patch.</div><div><br></div><div>I have a potential issue with "reasonable period". In the thread, someone mentioned "~3 hours" as the time to wait. But having brokenness for a good part of a business day is unhelpful even if it's only one thing that's broken at a given time. Regressions are introduced more frequently than one per three hours on average, so a grace period this long will result in never having green tests (here I assume that no one advocates for any sort of grace period for build failures).</div><div><br></div><div>My strong preference is immediate reaction. It doesn't always have to be a rollout, sometimes an issue can be fixed, or some tests can even be temporarily skipped - just make the tree green and stable for everyone else, as quickly as possible. But if the author is not available, and the bot watcher doesn't have a better fix (or is simply overwhelmed with multiple regressions being under investigation at once), I think that immediate rollout should be considered normal.</div><br><div>- Alexey</div></div></div></body></html>