Change in gerrit-admin[master]: Added hook to propagate CI flag

dcaro at redhat.com dcaro at redhat.com
Tue Dec 1 18:05:59 UTC 2015


David Caro has submitted this change and it was merged.

Change subject: Added hook to propagate CI flag
......................................................................


Added hook to propagate CI flag

Right now, if you create a non-code patch before getting feedback from
jenkins, that patch will never get CI review from it as jenkins will
review only the patchset that has code changes. This hook is triggered
on any comment, and propagates the CI value to the latest patchset if
it does not have a CI value already.

Change-Id: I0b6f3a53edd0a8d2a0114fb728fff0054c11d466
Signed-off-by: David Caro <dcaroest at redhat.com>
---
A hooks/custom_hooks/comment-added.propagate_review_values
M hooks/hook-dispatcher
M hooks/lib/gerrit.py
M hooks/lib/tools.py
4 files changed, 320 insertions(+), 47 deletions(-)

Approvals:
  David Caro: Verified; Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/48518
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0b6f3a53edd0a8d2a0114fb728fff0054c11d466
Gerrit-PatchSet: 16
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaro at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: David Caro <dcaro at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Max Kovgan <mvk at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Sagi Shnaidman <sshnaidm at redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaftaly at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list