Integrated: 2556: LablerWorkItem adds unnecessary overhead when the bot restarts

Zhao Song zsong at openjdk.org
Thu Aug 7 16:17:26 UTC 2025


On Wed, 6 Aug 2025 21:11:22 GMT, Zhao Song <zsong at openjdk.org> wrote:

> In [SKARA-2408](https://bugs.openjdk.org/browse/SKARA-2408) and [SKARA-2552](https://bugs.openjdk.org/browse/SKARA-2552), I updated the bot to do one more CheckWorkItem with force update after a pr is marked as auto labeled.
> 
> Currently, when the pr bot restarts, the bot would schedule a PullRequestCommandWorkItem for every pr(including closed prs), in the end of PullRequestCommandWorkItem, a LabelerWorkItem will be scheduled for each pr as well. In the LabelerWorkItem, when it finds the bot doesn't have label configuration, the bot will mark the pr as auto labelled and schedule a CheckWorkItem with forceUpdate. This is the issue, the bot shouldn't schedule any CheckWorkItem when the bot doesn't have label configuration. Otherwise, the bot would re-evaluate all the prs with forceUpdate.
> 
> Also, if the pr is already closed, the LabelerWorkItem shouldn't schedule any CheckWorkItem for it.

This pull request has now been integrated.

Changeset: becf9411
Author:    Zhao Song <zsong at openjdk.org>
URL:       https://git.openjdk.org/skara/commit/becf941128bca39facba3e30723f031f700f8ab6
Stats:     15 lines in 2 files changed: 8 ins; 0 del; 7 mod

2556: LablerWorkItem adds unnecessary overhead when the bot restarts

Reviewed-by: erikj

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

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


More information about the skara-dev mailing list