RFR: 2189: Mark command as handled when JCheckConfiguration is missing or invalid in PullRequestCommandWorkItem
Zhao Song
zsong at openjdk.org
Fri Mar 1 23:34:08 UTC 2024
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.
-------------
Commit messages:
- SKARA-2189
Changes: https://git.openjdk.org/skara/pull/1616/files
Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1616&range=00
Issue: https://bugs.openjdk.org/browse/SKARA-2189
Stats: 60 lines in 2 files changed: 56 ins; 1 del; 3 mod
Patch: https://git.openjdk.org/skara/pull/1616.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1616/head:pull/1616
PR: https://git.openjdk.org/skara/pull/1616
More information about the skara-dev
mailing list