Account for ignorable payloads in retry script, try to prevent them in runner

mozilla / jira-bugzilla-integration

Jira Bugzilla Integration (JBI) - system to sync bugs and issues

Mozilla Public License 2.0

8 stars 22 forks source link

Account for ignorable payloads in retry script, try to prevent them in runner #1000

Closed grahamalama closed 1 month ago

grahamalama commented 1 month ago

Somehow, some payloads that we'd normally ignore ended up in our Dead Letter Queue. This PR:

accounts for such payloads in the retry script when trying to process items in the queue
tries to prevent such items from making in into the queue by moving the guard to check if the bug has a matching a whiteboard tag before we refresh the bug data

grahamalama commented 1 month ago

@leplatrem I'm not sure why it was done that way. Seems like more of an oversight than anything

Any chance that the whiteboard field wouldn't be available in private bugs or something like that?

The first thing we do in execute_action is check if the bug is private and bail out if it is, so we can assume it'll be a public bug with a whiteboard field