Change in gerrit-admin[master]: Adapted stable branches checks and python

dcaroest at redhat.com dcaroest at redhat.com
Wed Jun 4 18:40:34 UTC 2014


David Caro has submitted this change and it was merged.

Change subject: Adapted stable branches checks and python
......................................................................


Adapted stable branches checks and python

- Showing contact address when login error occurs
- Adapted to new login procedures from bugzilla updates

Bug-Url hook:
- Skiping bugs out of the ovirt-related products
- If the patch is in the stable branch:
 * -1 if no bug-url
 * -1 if any target release incorrect
 * -1 if any private bug

set_POST hook:
- Warning message when not oVirt product
- Warning message when incorrect TR for the branch

set_MODIFIED hook:
- Warning message when not oVirt product

Flow changes:
* Only one review per commit, done by the dispatcher
* Hooks must output the result on stdout and the results will be
  aggregated
* Added more python libraries
* New hook that updates the external tracker metadata
* Changed the directory layout

Change-Id: I233f50c77fc2d419a5bc29b2ee5b3299c4bd6293
Signed-off-by: David Caro <dcaroest at redhat.com>
---
A .gitignore
D hooks/bz/change-merged.bz.set_MODIFIED
D hooks/bz/patchset-created.bz.1.bug_url
D hooks/bz/patchset-created.bz.2.add_tracker
D hooks/bz/patchset-created.bz.2.set_POST
D hooks/bz/patchset-created.bz.99.review_ok
D hooks/bz/patchset-created.warn_if_not_merged_to_previous_branch
A hooks/custom_hooks/change-abandoned.update_tracker
A hooks/custom_hooks/change-merged.set_MODIFIED
A hooks/custom_hooks/change-merged.update_tracker
A hooks/custom_hooks/patchset-created.bz.0.has_bug_url
A hooks/custom_hooks/patchset-created.bz.1.is_public
A hooks/custom_hooks/patchset-created.bz.2.correct_product
A hooks/custom_hooks/patchset-created.bz.3.correct_target_release
A hooks/custom_hooks/patchset-created.bz.98.set_POST
A hooks/custom_hooks/patchset-created.bz.99.review_ok
A hooks/custom_hooks/patchset-created.update_tracker
A hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch
A hooks/custom_hooks/update_tracker
M hooks/hook-dispatcher
A hooks/lib/__init__.py
A hooks/lib/bz.py
M hooks/lib/bz.sh
M hooks/lib/conf.sh
M hooks/lib/config.py
M hooks/lib/gerrit.py
M hooks/lib/gerrit.sh
A hooks/lib/tools.py
M hooks/lib/tools.sh
29 files changed, 1,367 insertions(+), 507 deletions(-)

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



-- 
To view, visit http://gerrit.ovirt.org/19600
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I233f50c77fc2d419a5bc29b2ee5b3299c4bd6293
Gerrit-PatchSet: 13
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Ohad Basan <obasan at redhat.com>



More information about the Engine-commits mailing list