RFR: 2038: Don't post error message of invalid jcheck configuration when encountering UncheckedIOException

Erik Joelsson erikj at openjdk.org
Fri Sep 22 20:31:43 UTC 2023


On Fri, 22 Sep 2023 18:30:04 GMT, Zhao Song <zsong at openjdk.org> wrote:

> In CheckWorkItem, the bot would check whether this pull request has a valid jcheck configuration and if not, the bot would post an error message of invalid jcheck configuration in the pull request. 
> 
> Today, a user reported that he saw this error message in his pull request and he couldn't determine whether the pull request is good. After reading the log, I found that it's just a temporary glitch. The bot was trying to read jcheck configuration in GitHub but got HttpTimeoutException, so the bot posted the error message. But it recovers in the next try. 
> https://github.com/openjdk/panama-foreign/pull/893 
> 
> To sum, in the cases of some temporary glitches, we shouldn't post error message to pull request to confuse users.

Marked as reviewed by erikj (Lead).

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

PR Review: https://git.openjdk.org/skara/pull/1559#pullrequestreview-1640603915


More information about the skara-dev mailing list