From Eyal Edri <eedri@redhat.com>:
Eyal Edri has submitted this change and it was merged. Change subject: Handle ext tracker 'None' status in set_modified hook ...................................................................... Handle ext tracker 'None' status in set_modified hook The hook checks the external trackers (patches) of the bug, and according to the external tracker 'status' field it checks if the patch is open ('POST') or closed ('MERGED'). The problem is when the user adds manual patch by adding the patch number, the 'status' parameter remain 'None'. Adding some code to query gerrit (if the 'status' == 'None') to get the real status of the patch. Change-Id: I56d37381e7af07a431b69d44b695667ad6a7f9ad Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1235 Signed-off-by: Shlomi Ben-David <sbendavi@redhat.com> --- M hooks/custom_hooks/set_modified 1 file changed, 78 insertions(+), 32 deletions(-) Approvals: Shlomo Ben David: Verified Eyal Edri: 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/73695 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56d37381e7af07a431b69d44b695667ad6a7f9ad Gerrit-PatchSet: 3 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Shlomo Ben David <sbendavi@redhat.com> Gerrit-Reviewer: Eyal Edri <eedri@redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shlomo Ben David <sbendavi@redhat.com>