Integrated: 2454: Skara mistakenly marked a non-clean backport as clean
Zhao Song
zsong at openjdk.org
Thu Mar 13 21:04:20 UTC 2025
On Fri, 7 Mar 2025 22:33:22 GMT, Zhao Song <zsong at openjdk.org> wrote:
> A user reported that Skara bot mistakenly marked a non-clean backport as clean. After investigation, I found that the PR was too large for the remote host to return the entire diff. The PR contained 8082 changed files, but when Skara bot requested the diff, only 3000 files were returned. Therefore, Skara bot was unable to compare some patches, so mistakenly mark the pr as clean.
>
> In this patch, Skara bot will now check if the PR is too large to evaluate. If it is, Skara bot will not evaluate if this PR is clean and will add a warning comment to the PR instead.
This pull request has now been integrated.
Changeset: 79fda799
Author: Zhao Song <zsong at openjdk.org>
URL: https://git.openjdk.org/skara/commit/79fda7998c04568620f1d4df40ff69e35b101066
Stats: 195 lines in 11 files changed: 166 ins; 3 del; 26 mod
2454: Skara mistakenly marked a non-clean backport as clean
Reviewed-by: erikj
-------------
PR: https://git.openjdk.org/skara/pull/1708
More information about the skara-dev
mailing list