<div dir="ltr">Why not use +1 on verified? That way the patch owner can wait till the code review process is over, mark it as verified, wait for CI and then submit.<div>It doesn&#39;t really give much added value to the code reviewers whether it&#39;s marked as verified or not</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 20, 2016 at 10:26 PM, Sandro Bonazzola <span dir="ltr">&lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><p dir="ltr"></p>
<p dir="ltr">Il 20/Nov/2016 17:25, &quot;Nir Soffer&quot; &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt; ha scritto:<br>
&gt;<br>
&gt; On Sun, Nov 20, 2016 at 5:39 PM, Yedidyah Bar David &lt;<a href="mailto:didi@redhat.com" target="_blank">didi@redhat.com</a>&gt; wrote:<br>
&gt; &gt; On Sun, Nov 20, 2016 at 5:06 PM, Barak Korren &lt;<a href="mailto:bkorren@redhat.com" target="_blank">bkorren@redhat.com</a>&gt; wrote:<br>
&gt; &gt;&gt; Hi all,<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Perhaps the main purpose of CI, is to prevent braking code from<br>
&gt; &gt;&gt; getting merged into the stable/master branches. Unfortunately our CI<br>
&gt; &gt;&gt; is not there yet, and one of the reasons for that is that we do large<br>
&gt; &gt;&gt; amount of our CI tests only _after_ the code is merged.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; The reason for that is that when balancing through, but time<br>
&gt; &gt;&gt; consuming, tests (e.g. enging build with all permutations) v.s. faster<br>
&gt; &gt;&gt; but more basic ones (e.g. &quot;findbugs&quot; and single permutation build), we<br>
&gt; &gt;&gt; typically choose the faster tests to be run per-patch-set and leave<br>
&gt; &gt;&gt; the through testing to only be run post-merge.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; We&#39;d like to change that and have the through tests also run before<br>
&gt; &gt;&gt; merge. Ideally we would like to just hook stuff to the &quot;submit&quot;<br>
&gt; &gt;&gt; button, but Gerrit doesn&#39;t allow one to do that easily. So instead<br>
&gt; &gt;&gt; we&#39;ll need to adopt some kind of flag to indicate we want to submit<br>
&gt; &gt;&gt; and have Jenkins<br>
&gt; &gt;&gt; &quot;click&quot; the submit button on our behalf if tests pass.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; I see two options here:<br>
&gt; &gt;&gt; 1. Use Code-Review+2 as the indicator to run &quot;heavy&quot; CI and merge.<br>
&gt;<br>
&gt; This is problematic. For example in vdsm we have 5 maintainers with<br>
&gt; +2, and 4 maintainers with commit right, but only 2 are commenting<br>
&gt; regularly.<br>
&gt;<br>
&gt; &gt;&gt; 2. Add an &quot;approve&quot; flag that maintainers can set to +1 (This is<br>
&gt; &gt;&gt;    what OpenStack is doing).<br>
&gt;<br>
&gt; This seems better.<br>
&gt;<br>
&gt; But there is another requirement - maintainer should be able to commit<br>
&gt; even if jenkins fails. Sometimes the CI is broken, or there are flakey tests<br>
&gt; breaking the build, and some jobs are failing regularly (check-merged)<br>
&gt; and I don&#39;t want to wait for it.</p>
</div></div><p dir="ltr">Either disable the jobs or fix them. Having jobs consitently failing and just ignore them is just a waste of resources. <br><br></p><div class="HOEnZb"><div class="h5">
<p dir="ltr">&gt;<br>
&gt; Today we can override the CI vote and commit, if we keep it as is I don&#39;t<br>
&gt; see any problem with this change.<br>
&gt;<br>
&gt; Nir<br>
&gt; ______________________________<wbr>_________________<br>
&gt; Devel mailing list<br>
&gt; <a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
&gt; <a href="http://lists.ovirt.org/mailman/listinfo/devel" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></p>
</div></div><br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br></div>