<div dir="ltr">adding also infra team for visibility on the change in CI.<div>also inline.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 11, 2015 at 4:19 PM, Francesco Romani <span dir="ltr">&lt;<a href="mailto:fromani@redhat.com" target="_blank">fromani@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">----- Original Message -----<br>
&gt; From: &quot;Yaniv Bronheim&quot; &lt;<a href="mailto:ybronhei@redhat.com">ybronhei@redhat.com</a>&gt;<br>
&gt; To: <a href="mailto:devel@ovirt.org">devel@ovirt.org</a>, &quot;Francesco Romani&quot; &lt;<a href="mailto:fromani@redhat.com">fromani@redhat.com</a>&gt;, &quot;Nir Soffer&quot; &lt;<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>&gt;, &quot;Piotr Kliczewski&quot;<br>
&gt; &lt;<a href="mailto:pkliczew@redhat.com">pkliczew@redhat.com</a>&gt;<br>
&gt; Cc: &quot;danken&quot; &lt;<a href="mailto:danken@redhat.com">danken@redhat.com</a>&gt;, &quot;David Caro&quot; &lt;<a href="mailto:dcaroest@redhat.com">dcaroest@redhat.com</a>&gt;, &quot;Eyal Edri&quot; &lt;<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>&gt;<br>
&gt; Sent: Thursday, December 10, 2015 6:46:37 PM<br>
&gt; Subject: Automation CI for vdsm<br>
<br>
[...]<br>
<span class="">&gt; We want to allow developers to trigger the script once reviews and<br>
&gt; verification are ready (last step before merge). To do so we agreed to add<br>
&gt; Continues Integration flag for each vdsm patch. </span></blockquote><div><br></div><div>This flag will be called &#39;Workflow&#39; or we can name it otherwise, we just need to choose what makes sense. </div><div>David/Yaniv - Please correct me if I&#39;m wrong.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Once this flag will be<br>
&gt; signed with +1 it will trigger Jenkins CI to run the check-merged script<br>
&gt; (adding new button to gerrit is not an option - you can image that flag as<br>
&gt; a trigger button), on success Jenkins CI flag will turn to +2. on fail<br>
&gt; we&#39;ll get -1 and once new patchset is ready the developer will remove the<br>
&gt; +1 and add it back to the Continues Integration flag to re-trigger the job.<br>
&gt;<br>
&gt; Please ack the process before we move on with that<br>
<br>
</span>Sounds good, even though I&#39;m a little scared (just gut feeling, no evidence<br>
whatsoever) that this could add even more complexity and fragility to the jenkins<br>
fleet.<br>
<br>
In the long run, when this is reliable, it will help greatly.<br>
In the short term, I&#39;m scared because this can lead to false positives and bogus<br>
failures.<br>
<br>
Let me stress I don&#39;t have concrete item to share or specific flaws.<br>
<br>
As action item on me, I will find some time next week to check virt functional tests,<br>
to see if they need some fixes, work reliably and so forth<br>
<span class=""><br>
&gt; The patch for those scripts still under review and testing -<br>
&gt; <a href="https://gerrit.ovirt.org/#/c/48268" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48268</a><br>
<br>
</span>Will review asap.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Francesco Romani<br>
RedHat Engineering Virtualization R &amp; D<br>
Phone: 8261328<br>
IRC: fromani<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Eyal Edri<br>Supervisor, RHEV CI<br>EMEA ENG Virtualization R&amp;D<br>Red Hat Israel<br><br>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div>
</div></div>