clusterfuzz icon indicating copy to clipboard operation
clusterfuzz copied to clipboard

Update jira to only add watchers that are missing

Open qyeo-sc opened this issue 3 weeks ago • 2 comments

There are two issues that this PR fixes:

  1. When self.client.add_watcher(issue.jira_issue, watcher) fails previously, it'd throw an exception resulting in the entire .save or .create call to fail, which would retry continuously hammering the jira instance.
  2. The function would always try to add watchers to issues regardless if the watchers were already added previously.

qyeo-sc avatar Dec 09 '25 19:12 qyeo-sc

@jonathanmetzman would you be able to take a look whenever you have the time?

qyeo-sc avatar Dec 09 '25 19:12 qyeo-sc

Is there any unit tests that can be added ?

@javanlacerda added some unit tests here

qyeo-sc avatar Dec 09 '25 23:12 qyeo-sc

@javanlacerda could you take another look whenever you have the chance?

qyeo-sc avatar Dec 16 '25 00:12 qyeo-sc