<p dir="ltr"><br>
Le 30 janv. 2016 09:32, "Julien Danjou" <<a href="mailto:julien@danjou.info">julien@danjou.info</a>> a écrit :<br>
><br>
> On Fri, Jan 29 2016, Sylvain Bauza wrote:<br>
><br>
> > While my heart is about that, my brain thinks about some regressions could<br>
> > be happening because of a +W even for a small change.<br>
><br>
> I suggest you read the git-revert manpage then, you might discover<br>
> something interesting there. :)<br>
></p>
<p dir="ltr">I suggest you look how to revert an RPC API change by thinking of our continuous deployers, you might discover something interesting there. :)</p>
<p dir="ltr">> The "shit happened" (e.g. bad thing merged) rate difference between a<br>
> "permission" policy and a "forgiveness" policy is based on my very<br>
> precise guessed estimation probably close to +1% in disfavor of<br>
> "forgiveness". Right.<br>
></p>
<p dir="ltr">I would like to understand your 1% estimate. Do you think that only one merged change is bad vs. 100 others good ?<br>
If so, how can you be sure that having an expert could not avoid the problem ?</p>
<p dir="ltr">> But at the same time, the velocity rate difference is close to +50% for<br>
> that same policy. So I've picked my side. :)<br>
></p>
<p dir="ltr">I disagree with you. Say that one change will raise an important gate issue if merged.<br>
Of course the change looks good. It's perfectly acceptable from a python perspective and Jenkins is happy.<br>
Unfortunately, merging that change would create lots of problems because it would wedge all the service projects CIs because that would be a behavioral change that wouldn't have a backwards compatibility.</p>
<p dir="ltr">If we have your forgiveness policy, it could have this change merged earlier, sure. But wouldn't you think that all the respective service projects velocities would be impacted by far more than this single change ?</p>
<p dir="ltr">-Sylvain</p>
<p dir="ltr">> --<br>
> Julien Danjou<br>
> ;; Free Software hacker<br>
> ;; <a href="https://julien.danjou.info">https://julien.danjou.info</a><br>
</p>