<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 30, 2016 at 11:45 AM, Roy Golan <span dir="ltr"><<a href="mailto:rgolan@redhat.com" target="_blank">rgolan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div><div><div>Hi all,<br><br></div><div>Eyal and I sat together to analyse and tweak the engine CI and this is what we came up with:<br></div><div><br></div>- dao test excludes updated - exclude dal/src/main/bundles from invoking dao tests<br></div> A trivial update of validation messages without any db change would trigger dao test without any need. This is one less job run for lots of patches.<br>This change is effective now.<br><br></div>- Spare CI re-run on trivial rebases <br></div> Gerrit trigger supports suppressing itself if the change to the tree was trivial. Most of the waste of resources(time and IO :) ) is around rebasing a change and waiting for CI to rerun. if Change1 is ci+1 and Change2 is ci+1 the chances that they will break CI together is very small and taking that risk is most probably worth it due to the huge resources waste <br></div> This change isn't effective yet - *Please reply here* if you agree or not to make this change available.<br></div></div></div></div></blockquote><div><br></div><div>+1, I also think the gain here is huge both for CI resources and for developers having to rebase multiple times and wait more time to merge their patches.</div><div>Even if there is a risk of 2 patches dependent and we'll fail on post merge once in a while, I think it outweight the alternative by far with benefits of resources & time. </div><div>(If we'll see too many failures we can always revert the change).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><br></div>All of this is 'master' - 3.6 will follow if we will agree on activating that change.<br><br></div>Thanks,<br></div>Roy<br></div>
<br>_______________________________________________<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/mailman/listinfo/devel</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Eyal Edri<br>Associate Manager</div><div>RHEV DevOps<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div>
</div></div>