[webkit-dev] Please avoid rolling out patches speculatively and reland them ASAP if you had to
Oliver Hunt
oliver at apple.com
Tue Dec 11 14:20:58 PST 2012
On Dec 11, 2012, at 2:17 PM, Peter Kasting <pkasting at chromium.org> wrote:
> On Tue, Dec 11, 2012 at 2:14 PM, Oliver Hunt <oliver at apple.com> wrote:
> I don't understand why anyone is _speculatively_ rolling out patches.
>
> You should only be rolling it out if you _know_ the patch is bad.
>
> Sometimes something bad happens to the tree, the sheriff doesn't know which patch is responsible, and the change authors are not present to ask for help. In a case like this the sheriff has to either do speculative rollouts or leave the tree broken.
>
> Ideally, of course, change authors are around when something like this happens. But maybe the bustage doesn't happen until much later, due to some subtle/latent issue, or maybe the change author is in fact irresponsible.
Or the sheriff could actually see if rolling out a patch locally fixes the problem. I'm not sure why they're considering "not testing" to be a valid behaviour for someone who is ostensibly meant to be keeping things going in the face of people who aren't testing.
>
> PK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20121211/be52db02/attachment.html>
More information about the webkit-dev
mailing list