[ovirt-devel] [ATN] [ACTION REQUIRED] Tweaking engine CI flow

Eyal Edri eedri at redhat.com
Wed Mar 30 08:51:10 UTC 2016


On Wed, Mar 30, 2016 at 11:45 AM, Roy Golan <rgolan at redhat.com> wrote:

> Hi all,
>
> Eyal and I sat together to analyse and tweak the engine CI and this is
> what we came up with:
>
> - dao test excludes updated - exclude dal/src/main/bundles from invoking
> dao tests
>   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.
> This change is effective now.
>
> - Spare CI re-run on trivial rebases
>   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
>   This change isn't effective yet - *Please reply here* if you agree or
> not to make this change available.
>

+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.
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.
(If we'll see too many failures we can always revert the change).


>
> All of this is 'master' - 3.6 will follow if we will agree on activating
> that change.
>
> Thanks,
> Roy
>
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>



-- 
Eyal Edri
Associate Manager
RHEV DevOps
EMEA ENG Virtualization R&D
Red Hat Israel

phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20160330/d26fdea9/attachment.html>


More information about the Infra mailing list