Integrated: 2247: When pr's headHash is missing, stop further processing
Zhao Song
zsong at openjdk.org
Tue Apr 30 16:05:14 UTC 2024
On Mon, 29 Apr 2024 22:01:27 GMT, Zhao Song <zsong at openjdk.org> wrote:
> Kevin Rushforth found this GitLab issue few years ago.
>
> Sometimes, pull requests in GitLab would be stuck. When it happens, it shows the number of commits as "0". Clicking on the "Changes" tab shows no diffs, and displays a warning banner: "Something went wrong on our end. Please try again!" This requires action on user's part to get them "unstuck".
>
> In Skara side, when it happens, pull request bot would not be able to get the headHash of this pull request and then throw an exception and trigger an other round of CheckWorkItem. If the user doesn't fix it, skara bot will continuously work on the pull request.
>
> To resolve it, we should try to make pull request bot be able to identify this situation and provide some guidance to the users.
This pull request has now been integrated.
Changeset: 695dbc6b
Author: Zhao Song <zsong at openjdk.org>
URL: https://git.openjdk.org/skara/commit/695dbc6b9603b5c1b4237ce24d4aacf9d1461d1d
Stats: 15 lines in 1 file changed: 15 ins; 0 del; 0 mod
2247: When pr's headHash is missing, stop further processing
Reviewed-by: erikj
-------------
PR: https://git.openjdk.org/skara/pull/1645
More information about the skara-dev
mailing list