RFR: 1042: Watchdog causing multiple restarts for mlbridge

[Kevin Rushforth]

On Fri, 14 May 2021 20:25:27 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.

Looks good.

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

Marked as reviewed by kcr (Committer).

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