Integrated: 1063: Lower log severity on HTTP errors

[Erik Joelsson]

On Wed, 2 Jun 2021 23:12:32 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> The Skara bots currently log all HTTP errors as SEVERE. With the log configuration we use, this results in admin alarms being sent whenever this happens (with some throttling). Now that we have metrics, we should instead leverage those and rely on alarms from our metrics gathering system instead. Typically HTTP errors aren't severe on their own, but if we get a lot of them, across multiple bot runners, then we need to react.
> 
> This patch does the following:
> 
> 1. Introduces a new specific exception which the RestRequest class throws when a bad HTTP response code is received.
> 2. Changes the logging behavior in the BotRunner when this new exception is caught.
> 3. Improves the logging of bad RestRequests so we actually log the request (instead of just the class name of the RequestBuilder).
> 4. Adds a new label on the skara_http_response counter to track if an error code was handled or not. We have a defined way of handling a bad HTTP response, and if this was used, we shouldn't send alarms as it's expected behavior.
> 5. Applies the http counters in two places where they were currently missing.

This pull request has now been integrated.

Changeset: 8e3c563a
Author:    Erik Joelsson <erikj at openjdk.org>
URL:       https://git.openjdk.java.net/skara/commit/8e3c563aea20761a350c997552dbde577f865517
Stats:     43 lines in 3 files changed: 37 ins; 1 del; 5 mod

1063: Lower log severity on HTTP errors

Reviewed-by: kcr

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

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