Integrated: 1673: CSR bot should be able to handle a withdrawn CSR properly

[Zhao Song]

On Thu, 10 Nov 2022 19:20:41 GMT, Zhao Song <zsong at openjdk.org> wrote:

> A user reported that skara bot keeps changing the description in his MR.
> 
> After investigation, we found a bug related with CSR bot.
> 
> The user linked a CSR issue with the main issue, and after that, he found CSR unneeded and withdrawn the CSR issue.
> 
> However, our logic about handling withdrawn CSR issue has some problem.
> 
> In PullRequestWorkItem#run, the bot would add updateMarker() to the PR body regardless of the state of CSR issue.  And if a PR body contains the updateMarker, it will be updated periodically until the body contains a CSR progress. However, since our CSR issue is withdrawn, the CSR label would not be added to the pr and CSR progress would not be added to the PR body. So it will be an endless loop.
> 
> In summary, the bug would happen in such a case, the user withdraw the csr issue before the csr issue bot first time run.
> 
> In this patch, before the CSR Issue bot is trying to add the updateMarker, it will check the resolution of the CSR Issue first, if the CSR Issue is already withdrawn, then updateMarker would not be added to the PR body.  And PR bot would work normally.

This pull request has now been integrated.

Changeset: fc863a29
Author:    Zhao Song <zsong at openjdk.org>
URL:       https://git.openjdk.org/skara/commit/fc863a29a7b21f04d9e133c0f7d75a47e24f2940
Stats:     66 lines in 2 files changed: 65 ins; 0 del; 1 mod

1673: CSR bot should be able to handle a withdrawn CSR properly

Reviewed-by: erikj

-------------

PR: https://git.openjdk.org/skara/pull/1418