Skip to content

Commit

Permalink
Register thanos_alert_sender_errors_total metric. (#2101)
Browse files Browse the repository at this point in the history
* Register thanos_alert_sender_errors_total metric.

#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>
  • Loading branch information
OGKevin committed Feb 5, 2020
1 parent 7c02430 commit 57a1eae
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Expand Up @@ -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

Expand Down
2 changes: 1 addition & 1 deletion pkg/alert/alert.go
Expand Up @@ -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
}
Expand Down

0 comments on commit 57a1eae

Please sign in to comment.