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


Devel mailing list

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

phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)