From 57a1eae62a94f55414f3c47d3bacb88f3c2a4732 Mon Sep 17 00:00:00 2001 From: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com> Date: Wed, 5 Feb 2020 14:38:59 +0100 Subject: [PATCH] Register thanos_alert_sender_errors_total metric. (#2101) * Register thanos_alert_sender_errors_total metric. https://github.com/thanos-io/thanos/issues/2100 Fixes #2100 Signed-off-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com> * Updated CHANGELOG. Signed-off-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com> --- CHANGELOG.md | 1 + pkg/alert/alert.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index dee286e74c..2f7a8d9066 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -16,6 +16,7 @@ We use *breaking* word for marking changes that are not backward compatible (rel - [#2033](https://github.com/thanos-io/thanos/pull/2033) minio-go: Fixed Issue #1494 support Web Identity providers for IAM credentials for AWS EKS - [#1985](https://github.com/thanos-io/thanos/pull/1985) store gateway: Fixed case where series entry is larger than 64KB in index. - [#2051](https://github.com/thanos-io/thanos/pull/2051) ruler: Fixed issue where ruler does not expose shipper metrics. +- [#2101](https://github.com/thanos-io/thanos/pull/2101) ruler: Fixed bug where thanos_alert_sender_errors_total was not registered. ### Added diff --git a/pkg/alert/alert.go b/pkg/alert/alert.go index 7cb11a75bd..418cb5c831 100644 --- a/pkg/alert/alert.go +++ b/pkg/alert/alert.go @@ -313,7 +313,7 @@ func NewSender( }, []string{"alertmanager"}), } if reg != nil { - reg.MustRegister(s.sent, s.dropped, s.latency) + reg.MustRegister(s.sent, s.errs, s.dropped, s.latency) } return s }