Change in gerrit-admin[master]: Fix Bug Url match

Code Review gerrit at ovirt.org
Mon Nov 28 14:41:15 UTC 2016


>From Shlomo Ben David <sbendavi at redhat.com>:

Shlomo Ben David has submitted this change and it was merged.

Change subject: Fix Bug Url match
......................................................................


Fix Bug Url match

Currently the check_bug_url checks only urls in the following format:
http://bugzilla.redhat.com/<bug-id>

This patch is fixing that and allows to catch urls with this format as well:
http://bugzilla.redhat.com/show_bug.cgi?id=<bug-id>

Change-Id: I9ca005de77c30e8860a5d7fd52fdd932ddac9afe
Fixes: OVIRT-843
Signed-off-by: Shlomi Ben-David <sbendavi at redhat.com>
---
M hooks/custom_hooks/check_bug_url
M hooks/lib/bz.py
2 files changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Shlomo Ben David: Verified; Looks good to me, approved
  Gil Shinar: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9ca005de77c30e8860a5d7fd52fdd932ddac9afe
Gerrit-PatchSet: 1
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>



More information about the Engine-commits mailing list