<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 18 Aug 2016, at 09:32, Sandro Bonazzola <<a href="mailto:sbonazzo@redhat.com" class="">sbonazzo@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Thu, Aug 18, 2016 at 9:19 AM, Michal Skrivanek <span dir="ltr" class=""><<a href="mailto:michal.skrivanek@redhat.com" target="_blank" class="">michal.skrivanek@redhat.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><br class=""><div class=""><div class=""><div class="h5"><blockquote type="cite" class=""><div class="">On 18 Aug 2016, at 09:09, Sandro Bonazzola <<a href="mailto:sbonazzo@redhat.com" target="_blank" class="">sbonazzo@redhat.com</a>> wrote:</div><br class=""><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Wed, Aug 17, 2016 at 5:12 PM, Nir Soffer <span dir="ltr" class=""><<a href="mailto:nsoffer@redhat.com" target="_blank" class="">nsoffer@redhat.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Aug 17, 2016 at 2:47 PM, Eyal Edri <<a href="mailto:eedri@redhat.com" target="_blank" class="">eedri@redhat.com</a>> wrote:<br class="">
><br class="">
><br class="">
> On Wed, Aug 17, 2016 at 2:25 PM, Nir Soffer <<a href="mailto:nsoffer@redhat.com" target="_blank" class="">nsoffer@redhat.com</a>> wrote:<br class="">
>><br class="">
</span><span class="">>> On Wed, Aug 17, 2016 at 1:45 PM, Eyal Edri <<a href="mailto:eedri@redhat.com" target="_blank" class="">eedri@redhat.com</a>> wrote:<br class="">
>> > I still thinks its a very valuable hook and we are aware of the fact it<br class="">
>> > has<br class="">
>> > bugs, especially with patches on master branch and 4.0.<br class="">
>> ><br class="">
>> > Shlomi from the infra team is working on a solution for it as we speak<br class="">
>> > and<br class="">
>> > we hope to have a solution in the next few days, however it's not<br class="">
>> > trival to<br class="">
>> > test and requires setting up a staging env and improve loga for the<br class="">
>> > hooks<br class="">
>> > system.<br class="">
>><br class="">
>> How do you plan to solve this?<br class="">
>><br class="">
</span><span class="">>> Only the owner of the bug knows if the all the required patches are merged<br class="">
><br class="">
><br class="">
</span><span class="">> The authors should use Bug-Url on the main bug and related-to: on other<br class="">
> patches that are related.<br class="">
<br class="">
</span>This is not possible. Many times you need series of patches to fix a bug, and<br class="">
you the number of patches may change during development. You start with one<br class="">
patch, and later you find that you need another one, so all of them will have<br class="">
a bug url.<br class="">
<br class="">
Practically, you should expect that all patches in the series will<br class="">
have a bug-url.<br class="">
If the hook will change the bug incorrectly someone will have to fix this, and<br class="">
it is very unlikely that a developer will go to clean after the hook.<br class="">
<span class=""><br class="">
>> and backported to the correct repositories.<br class="">
><br class="">
><br class="">
</span><span class="">> This is done with logic according to the bug target milestone.<br class="">
><br class="">
> for e.g - a patch on branch 'ovirt-engine-4.0' was merged to bug targeted to<br class="">
> ovirt-4.0.2.<br class="">
> The hook should check if branch 4.0.2 exists or not, if it exists then the<br class="">
> bug should NOT move to MODIFIED,<br class="">
> since it needs still backporting to ovirt-engine-4.0.2 branch.<br class="">
<br class="">
</span>This is too fragile. For example, maybe a 4.0.2 branch is created after<br class="">
the patch was merged to 4.0 branch, and the patch will be missing, although<br class="">
the bug is already set to modified.<br class="">
<br class="">
Setting to modified should be done by the owner of the bug, after verifying<br class="">
that the patches exists in correct branch.<br class="">
<br class="">
I'm not suggesting to remove the hook, just disable the feature of making<br class="">
a bug modified.<br class=""></blockquote><div class=""><br class=""></div><div class="">+1. On build day checking that bugs in modified not listed in Bug-Url on the build branch due to missing backport is a painful experience.</div></div></div></div></div></blockquote><div class=""><br class=""></div></div></div>it is a tradeoff. It was mentioned before that the other way around we would be left with too many POSTed bugs which are actually already merged. The maintainer is typically not the assignee so if you e.g. merge the last patch on Thursday afternoon it takes some time to gets attention, in the meantime there is a build which won’t consider that bug.</div><div class=""><br class=""></div></div></blockquote><div class=""><br class=""></div><div class="">The other way around is having a modified bugs which should have been in post being considered in the build, moved to QE, failing QE, move back to assigned.</div><div class="">Not sure it's much better.</div></div></div></div></div></blockquote><div><br class=""></div>it is when the amount of false positive ON_QA bugs is far less than amount of forgotten bugs</div><div>I’m not advocating for it, I’m just saying it was pointed out that this was the situation before we introduced the hook.</div><div>Perhaps with the doc police emails it is no longer a relevant point</div><div><br class=""></div><div>Thanks,</div><div>michal</div><div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class=""></div><div class=""><br class=""></div><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""></div><div class=""><br class=""><blockquote type="cite" class=""><div class=""><span class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class=""></div><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><font color="#888888" class=""><br class="">
Nir<br class="">
</font></span></blockquote></div><br class=""><br clear="all" class=""><div class=""><br class=""></div>-- <br class=""><div data-smartmail="gmail_signature" class=""><div dir="ltr" class=""><div class=""><div dir="ltr" class="">Sandro Bonazzola<br class="">Better technology. Faster innovation. Powered by community collaboration.<br class="">See how it works at <a href="http://redhat.com/" target="_blank" class="">redhat.com</a><br class=""></div></div></div></div>
</div></div></span><span class="">
______________________________<wbr class="">_________________<br class="">Devel mailing list<br class=""><a href="mailto:Devel@ovirt.org" target="_blank" class="">Devel@ovirt.org</a><br class=""><a href="http://lists.ovirt.org/mailman/listinfo/devel" target="_blank" class="">http://lists.ovirt.org/<wbr class="">mailman/listinfo/devel</a></span></div></blockquote></div><br class=""></div></blockquote></div><br class=""><br clear="all" class=""><div class=""><br class=""></div>-- <br class=""><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr" class=""><div class=""><div dir="ltr" class="">Sandro Bonazzola<br class="">Better technology. Faster innovation. Powered by community collaboration.<br class="">See how it works at <a href="http://redhat.com/" target="_blank" class="">redhat.com</a><br class=""></div></div></div></div>
</div></div>
</div></blockquote></div><br class=""></body></html>