<p dir="ltr">What does it mean to clean the jobs?  You mean git history or temp files? </p>
<div class="gmail_extra"><br><div class="gmail_quote">On Nov 24, 2016 1:23 AM, &quot;Martin Perina&quot; &lt;<a href="mailto:mperina@redhat.com">mperina@redhat.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">So, I tested current master and also upgrade from 4.0 to master and everything works fine on my local machine.<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">But I&#39;ve found 2 issues:<br><br>1. Jenkins CI for some reason unknown to me doesn&#39;t use current master (change-id: I59087ff8902e35b9f7c922eba0a17<wbr>ba4f0791795) but it uses older commit (most probably revert patch from Eyal [1] with Change-Id: I8c2350aba48619dfa84624b343cfa<wbr>167d0b6e0ed).as a current master when executing upgrade job from master to patch<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">2. When we are reverting patches which contain db upgrade scripts which are not last (for example we are dropping db script 04_01_0500 when there&#39;s already merged script 04_01_0510) we cannot delete upgrade script otherwise db schema upgrade script fails (revert script [1] is faulty). Only following actions are valid in this case:<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">  a. Comment out code in db upgrade script we want to revert, but leave this script in the repo<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">  b. Remove the db upgrade script and renumber all later db upgrade scripts so there&#39;s no hole in db upgrade script numbering<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">Conclusion: current code in master is OK, we need clean up jenkins job to really use latest merged patch on git master branch.<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default">Martin<br><br></div><div style="font-family:arial,helvetica,sans-serif" class="gmail_default"><br>[1] <a href="https://gerrit.ovirt.org/#/c/67205/" target="_blank">https://gerrit.ovirt.org/#/c/<wbr>67205/</a><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 23, 2016 at 7:17 PM, Michal Skrivanek <span dir="ltr">&lt;<a href="mailto:mskrivan@redhat.com" target="_blank">mskrivan@redhat.com</a>&gt;</span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><br>
<br>
&gt; On 23 Nov 2016, at 17:44, Martin Sivak &lt;<a href="mailto:msivak@redhat.com" target="_blank">msivak@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt; Yeah, our patches are still failing as well. It seems the patch was<br>
&gt; not enough or there is some other conflicting db script.<br>
<br>
I think the hash changed. IIRC there was some issue in the logic when the existing script is updated (since it was merged and reverted and re-merged with the same number, but it was not the exact same content)<br>
Martin/Eli to the rescue!<br>
<br>
&gt;<br>
&gt; Martin<br>
&gt;<br>
&gt;&gt; On Wed, Nov 23, 2016 at 2:16 PM, Sandro Bonazzola &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt; On Wed, Nov 23, 2016 at 2:13 PM, Arik Hadas &lt;<a href="mailto:ahadas@redhat.com" target="_blank">ahadas@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; it should be fixed now (by <a rel="noreferrer" href="https://gerrit.ovirt.org/#/c/67210/" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>7210/</a>)<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; I rebased <a rel="noreferrer" href="https://gerrit.ovirt.org/66999" target="_blank">https://gerrit.ovirt.org/66999</a> and still failing there.<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ----- Original Message -----<br>
&gt;&gt;&gt;&gt; engine master CI is failing on $subject, please fix or revert as soon as<br>
&gt;&gt;&gt;&gt; possible, thanks.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; --<br>
&gt;&gt;&gt;&gt; Sandro Bonazzola<br>
&gt;&gt;&gt;&gt; Better technology. Faster innovation. Powered by community<br>
&gt;&gt;&gt;&gt; collaboration.<br>
&gt;&gt;&gt;&gt; See how it works at <a rel="noreferrer" href="http://redhat.com" target="_blank">redhat.com</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; ______________________________<wbr>_________________<br>
&gt;&gt;&gt;&gt; Devel mailing list<br>
&gt;&gt;&gt;&gt; <a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
&gt;&gt;&gt;&gt; <a rel="noreferrer" href="http://lists.ovirt.org/mailman/listinfo/devel" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Sandro Bonazzola<br>
&gt;&gt; Better technology. Faster innovation. Powered by community collaboration.<br>
&gt;&gt; See how it works at <a rel="noreferrer" href="http://redhat.com" target="_blank">redhat.com</a><br>
&gt;&gt;<br>
&gt;&gt; ______________________________<wbr>_________________<br>
&gt;&gt; Devel mailing list<br>
&gt;&gt; <a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
&gt;&gt; <a rel="noreferrer" href="http://lists.ovirt.org/mailman/listinfo/devel" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><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 rel="noreferrer" href="http://lists.ovirt.org/mailman/listinfo/devel" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
&gt;<br>
&gt;<br>
</blockquote></div><br></div></div>
</blockquote></div></div>