Integrated: 2189: Mark command as handled when JCheckConfiguration is missing or invalid in PullRequestCommandWorkItem

[Zhao Song]

On Fri, 1 Mar 2024 23:11:57 GMT, Zhao Song <zsong at openjdk.org> wrote:

> Recently, we found that the skara bot keeps trying to process /open command in this pr https://github.com/openjdk/jdk8u-dev/pull/393
> 
> After investigation, we found that this pr is a dependent pr and this pr was closed before the parent pr(https://github.com/openjdk/jdk8u-dev/pull/392) got integrated, so this dependent pr didn't get retargeted.
> 
> Skara bot was trying to process the command but the bot couldn't find jcheck configuration in the target branch because the target is already deleted. And when the jcheck configuration is invalid or missing in PullRequestCommandWorkItem, RuntimeException would be thrown, causing the bot to repeatedly process the command. This behavior is wrong because when jcheck configuration is missing or invalid, it wouldn't recover automatically. In this case, the bot should just notice the user what problem it found and mark the command as handled to prevent the bot from being stuck in a loop.

This pull request has now been integrated.

Changeset: 7b937ab9
Author:    Zhao Song <zsong at openjdk.org>
URL:       https://git.openjdk.org/skara/commit/7b937ab96469ff0f9c8dcaa4c56f533b07028ba0
Stats:     68 lines in 2 files changed: 64 ins; 1 del; 3 mod

2189: Mark command as handled when JCheckConfiguration is missing or invalid in PullRequestCommandWorkItem

Reviewed-by: erikj

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

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