<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 25 June 2015 at 16:08, John Davidge (jodavidg) <span dir="ltr"><<a href="mailto:jodavidg@cisco.com" target="_blank">jodavidg@cisco.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div style="word-wrap:break-word;color:rgb(0,0,0);font-size:14px;font-family:Calibri,sans-serif">
<div>Hi all,</div>
<div><br>
</div>
<div>Recent neutron third party CI issues have got me thinking again about a topic which we discussed in Vancouver:</div>
<div><br>
</div>
<div>Should any Third Party CI have voting rights for neutron patches in gerrit?</div></div></blockquote><div><br></div><div>Why should this be a decision for Neutron only?</div><div> </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;color:rgb(0,0,0);font-size:14px;font-family:Calibri,sans-serif">
<div><br>
</div>
<div>I’d like to suggest that they shouldn’t.</div>
<div><br>
</div>
<div>A -1 from a third party CI tool can often be an indication that the CI tool itself or the third party plugin is broken, rather than there being issues with the patch under review. I don’t think there are many cases where a third party CI tool has caught
 a genuine issue that Jenkins has missed. With the current voting rights these CI tools cause a lot of noise when they experience problems.</div></div></blockquote><div><br></div><div>As far as I am aware no 3rd party CI tool has a better coverage than the upstream one.</div><div>some 3rd party CIs exercise different code paths and might uncover some issue that the upstream CI did not cover. There will surely be people claiming this has happened a lot of times, and even a single issue found is invaluable; I would agree with that, but I also think that a 3rd party CI does not have to vote to be useful.</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;color:rgb(0,0,0);font-size:14px;font-family:Calibri,sans-serif">
<div><br>
</div>
<div>I’m not suggesting that the results of these tests be removed from the page altogether - there are some cases where their results are useful to the patch author/reviewer - but removing voting rights (or at least -1 rights) would save a patch from a –1
 that might not be particularly meaningful.</div></div></blockquote><div><br></div><div>Frankly I find the overwhelming number of CI messages - and email notifications even more annoying that random -1s. Thankfully you can hide the formers and filter out the latters.</div><div>From the perspective of 3rd party CI maintainer I could use myself as an example; I maintain a CI which has now been broken for about 48 hours. I am busy with other tasks and cannot look at it now. I might be a terrible person for this, but that's my problem. If the CI was not voting at least I would not have annoyed people. (fwiw, I've disabled my CI now).<br></div><div><br></div><div>Also, I believe we already agreed that a working CI is not anymore a requirement, as long as the plugin/driver maintainers can provide a reasonable proof that their integration works?</div><div><br></div><div>Salvatore</div><div> </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;color:rgb(0,0,0);font-size:14px;font-family:Calibri,sans-serif">
<div><br>
</div>
<div>Thoughts?</div><span class="HOEnZb"><font color="#888888">
<div><br>
</div>
<div>John</div>
</font></span></div>

<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>