On Wed, Mar 30, 2016 at 11:45 AM, Roy Golan <rgolan(a)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(a)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)