Re: [Ksummit-discuss] bug-introducing patches

From: Willy Tarreau
Date: Thu May 03 2018 - 13:57:40 EST


On Thu, May 03, 2018 at 05:29:29PM +0000, Sasha Levin wrote:
> I tried pulling all the fixes that went in 4.17 (so far) for bugs that
> were introduced as fixes in the v4.16 cycle, I got this list:
>
> d65026c6c62e v4.16-rc7 5 6b1e6cc7855b v4.7 d14d2b78090c
> 63489f8e8211 v4.16-rc6 13 045c7a3f53d9 v4.11-rc6 5df63c2a149a
> 5dcd8400884c v4.16-rc6 6 0759e552bce7 v4.7 bd28899dd34f
> 0ef58b0a05c1 v4.16-rc6 6 0cf737808ae7 v4.14 a56d99d71466 7992894c305e 2afc5d61a719
> 8936ef7604c1 v4.16-rc6 6 6c8702c60b88 v4.9 a957fa190aa9
> bbc09e7842a5 v4.16-rc6 6 65a206c01e8e v4.13 3239534a79ee
> 6a2cf8d3663e v4.16-rc5 12 d64d6c5671db v4.15 6d6340672ba3
> 859d880cf544 v4.16-rc4 14 b68a68d3dcc1 v4.15 8420f71943ae
> e39a97353e53 v4.16-rc4 16 2a842acab109 v4.12 cbe095e2b584
> a27fd7a8ed38 v4.16-rc4 19 f214f915e7db v4.13 bffd168c3fc5
> 0f9da844d877 v4.16-rc2 16 28128c61e08e v4.16-rc2 a95b37e20db9
> 7324f5399b06 v4.16-rc2 19 186b3c998c50 v4.14 51568d69407d
> e78c637127ee v4.16-rc3 25 187d7967a5ee v4.4 e988867fd774
> ca9eee95a2de v4.16-rc3 25 d717f7352ec6 v4.12 e988867fd774
>
> So out of 755 commits, 14 have been fixed, that's about 2% and we're not
> even done with 4.17.

OK but this is low. Quite frankly, at 2% regressions, even if this is
never fun, it means 98% of the fixes were right. Now just delay them
longer and you'll have 500 commits instead of 755, thus 255 more bugs
unfixed in the release just to try to save 14 wrong ones. *this* is
the problem I'm concerned about by enforcing extra delays on everyone.
This is the reason why in my opinion the most important is to raise
awareness about this so people are more careful or more verbose (and
more detailed commit messages don't hurt, I think all stable maintainers
have many times thought "WTF is this supposed to fix?"), and then remind
everyone that when some get caught abusing, they'll get a public blame.

> >Some may get caught by your automated script if a later commit fixes
> >an issue introduced by their patch. And there public shaming is the
> >only option (or maybe only the second time if you really want to be
> >nice).
>
> I'd much prefer to blame this on maintainers. Authors should be able to
> submit a patch whenever they feel like it, maintainers should only merge
> a patch in when it's right.

Sorry, wrong word on my side, I also meant maintainers (I very much favor
pushing back to ensure everyone in the chain is responsible for what is
done).

Willy