<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 4, 2016 at 6:13 PM, Raghavendra Talur <span dir="ltr">&lt;<a href="mailto:rtalur@redhat.com" target="_blank">rtalur@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 dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, Feb 4, 2016 at 7:13 PM, Niels de Vos <span dir="ltr">&lt;<a href="mailto:ndevos@redhat.com" target="_blank">ndevos@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"><span>On Thu, Feb 04, 2016 at 04:15:16PM +0530, Raghavendra Talur wrote:<br>
&gt; On Thu, Feb 4, 2016 at 4:13 PM, Niels de Vos &lt;<a href="mailto:ndevos@redhat.com" target="_blank">ndevos@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt; On Thu, Feb 04, 2016 at 03:34:05PM +0530, Raghavendra Talur wrote:<br>
&gt; &gt; &gt; Hi,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; We recently changed the jenkins builds to be triggered on the following<br>
&gt; &gt; &gt; triggers.<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; 1. Verified+1<br>
&gt; &gt; &gt; 2. Code-review+2<br>
&gt; &gt; &gt; 3. recheck (netbsd|centos|smoke)<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; There is a bug in 1 and 2.<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Multiple triggers of 1 or 2 would result in re-runs even when not<br>
&gt; &gt; intended.<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; I would like to replace 1 and 2 with a comment &quot;run-all-regression&quot; or<br>
&gt; &gt; &gt; something like that.<br>
&gt; &gt; &gt; Thoughts?<br>
&gt; &gt;<br>
&gt; &gt; Maybe starting regressions on Code-Review +1 (or +2) only?<br>
&gt; &gt;<br>
&gt;<br>
&gt; Multiple code-reviews would do multiple triggers. Won&#39;t work.<br>
<br>
</span>How can we make this to work, without the need of providing magic<br>
comments?<br></blockquote><div><br></div></span><div>I investigated but couldn&#39;t find a way to make it work. Discussed with Kaushal and we feel it should be ok to go with a &quot;check all&quot; comment for initial regression run and deprecate Code-Review+2 and Verified+1 triggers.</div><div><br></div><div>I would like to go ahead and do it as the build queue is increasing again just because of Code-Review+2&#39;s given just before a patch is merged; they don&#39;t serve any purpose.</div></div></div></div></blockquote><div><br></div><div>I have for now just removed trigger for code-review.</div><div>Trigger for verified+1 remains as is.</div><div>No new trigger on comments have been added.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><font color="#888888"><br>
Niels<br>
</font></span></blockquote></div><br></div></div>
</blockquote></div><br></div></div>