RFR: 1042: Watchdog causing multiple restarts for mlbridge [v2]

[Erik Helin]

On Mon, 17 May 2021 17:44:43 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> When starting certain bots with a fresh scratch area, we currently end up in a restart loop. This is because all the threads immediately get busy cloning repos, which starves out the watchdog pings for longer than the hard coded 10 minutes. This patch changes the watchdog to use the configuration setting "watchdog" for the restart timeout instead. This value is currently used for a log warning which is also driven by the watchdog, so to be able to still have separate values, I've introduced a new option "watchdog_warn" which can optionally be set for just the warning part.
>> 
>> In addition to this, I also added a bit more logging to make it easier to follow through logstash when watchdog pings occur, or when a new instance of a bot runner is started. Failure to start due to configuration errors are now also posted using proper logs.
>
> Erik Joelsson has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Restored direct printing of error messages

Looks good!

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

Marked as reviewed by ehelin (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/1157