6 Commits (2bab11e20a60ddb9d7a43302efa50fbe09422a3e)

Author	SHA1	Message	Date
lpugoy	c792af3ad0	grafana-mixin: Fix expression for GrafanaRequestsFailing alert (#63382) ... Fix expression for GrafanaRequestsFailing alert The intent of the alert is to get the ratio of 5xx to all status codes [^1]. With the original expression, the left hand side can have more than one row with the same labels except for the status code. This results in a promql error because it is doing a many-to-one matching. Doing a sum on the left hand side first should preserve the intent of the alert and resolve the issue. [^1]: https://github.com/grafana/grafana/pull/43116	2 years ago
Carlos Juan Gómez Peñalver	984e293d60	many-to-one requires explicit grouping (#58174) ... The current query results in a many-to-one relationship which requires explicit grouping see more details here https://github.com/portefaix/portefaix-hub/pull/306 https://prometheus.io/docs/prometheus/latest/querying/operators/#many-to-one-and-one-to-many-vector-matches	2 years ago
Pedro Araújo	18fdb89554	grafana-mixin: Fix `GrafanaRequestsFailing` alert (#43116) ... Due to PromQL's label matching, both sides of the division will have same series; that means that, whenever there's a 5xx error, both sides will have the same value and the division will be `1`. I believe the idea was to get the ratio of 5xx compared will all status code, and to do that, we need to aggregate the `status_code` dimension away.	3 years ago
Carl Bergquist	075256923b	Update the grafana-mixin to use HTTP histograms (#39155) ... Signed-off-by: bergquist <carl.bergquist@gmail.com>	4 years ago
Carl Bergquist	cf6512abd9	reduce severity level to warning (#28939) ... Signed-off-by: bergquist <carl.bergquist@gmail.com>	5 years ago
Carl Bergquist	6002df580f	Add monitoring mixing for Grafana (#28285) ... Co-authored-by: Tom Wilkie <tom.wilkie@gmail.com>	5 years ago