Patch statuses are not updated in Bugzilla

Description

When patch is merged in gerrit, it's status is not updated in Bugzilla and patch stays in POST state.

For example, patch 80617 [1] was merged today, but it is still in a POST state at bugzilla [2]

[1] https://gerrit.ovirt.org/#/c/80617/
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1445199

Activity

Show:

Former user September 18, 2017 at 2:55 PM

Hi Eyal,

patch #81877 that solves it is ready and waiting for CR.
It looks like there was some changes in Bugzilla that returns the external tracker id number as string instead of int as it was previously.

Eyal Edri September 17, 2017 at 11:01 AM

did you have a chance to look into this issue?

Sandro Bonazzola September 4, 2017 at 7:41 AM

I've seen this too.

Former user September 1, 2017 at 1:59 PM

Gerrit was updated recently as part of https://ovirt-jira.atlassian.net/browse/OVIRT-1450#icft=OVIRT-1450 so this might be related.
Looking at In logs on Gerrit I see the hook triggered successfully:

2017-09-01 09:25:33,718::16918::INFO::change-merged.bz.99.update_tracker
2017-09-01 09:25:30,646::16923:grinning face with big eyesEBUG::ESC[94m----------------------------------------------ESC[0m
2017-09-01 09:25:30,646::16923:grinning face with big eyesEBUG::ESC[96mSTART HOOK: change-merged.bz.99.update_trackerESC[0m
2017-09-01 09:25:30,647::16923:grinning face with big eyesEBUG::ESC[94m----------------------------------------------ESC[0m
...
2017-09-01 09:25:30,874::16923:grinning face with big eyesEBUG::==> bug_urls: [u'https://bugzilla.redhat.com/1445199']
2017-09-01 09:25:30,874::16923:grinning face with big eyesEBUG::==> bug_ids: set([u'1445199'])

2017-09-01 09:25:30,874::16923:grinning face with big eyesEBUG::==> updating tracker with bug_id: 1445199

2017-09-01 09:25:30,874::16923:grinning face with big eyesEBUG::==> checking bug_id: 1445199
2017-09-01 09:25:31,952::16923:grinning face with big eyesEBUG::==> classification: oVirt
2017-09-01 09:25:31,953::16923:grinning face with big eyesEBUG::==> product: ovirt-engine
2017-09-01 09:25:33,699::16923:grinning face with big eyesEBUG::ESC[94m--------------------------------------------ESC[0m
2017-09-01 09:25:33,699::16923:grinning face with big eyesEBUG::ESC[96mEND HOOK: change-merged.bz.99.update_trackerESC[0m
2017-09-01 09:25:33,699::16923:grinning face with big eyesEBUG::ESC[94m--------------------------------------------ESC[0m

2017-09-01 09:25:33,719::16918:grinning face with big eyesEBUG::==> OUTPUT::
################################################################################
0
0

  • Update Tracker::#1445199::OK, tracker status: MERGED
    ################################################################################

2017-09-01 09:25:33,719::16918::INFO::==> RESULT::{'msg': '* Update Tracker::#1445199::OK, tracker status: MERGED', 'code_review': 0, 'verified': 0}

The last line is also saved in gerrit UI, so something in Gerrit-Bugzilla integration is broken as the hook thinks the tracker was updated while in fact it wasn't

Fixed

Details

Assignee

Reporter

Components

Priority

Created September 1, 2017 at 1:36 PM
Updated November 1, 2017 at 10:02 AM
Resolved October 15, 2017 at 2:52 PM

Flag notifications