-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vmalert: support configuration file for notifiers #2127
Conversation
* vmalert notifiers now can be configured via file see https://docs.victoriametrics.com/vmalert.html#notifier-configuration-file * add support of Consul service discovery for notifiers config see #1947 * add UI section for currently loaded/discovered notifiers * deprecate `-rule.configCheckInterval` in favour of `-configCheckInterval` Signed-off-by: hagen1778 <roman@victoriametrics.com>
Codecov Report
@@ Coverage Diff @@
## master #2127 +/- ##
==========================================
+ Coverage 60.31% 60.51% +0.20%
==========================================
Files 240 239 -1
Lines 40269 40279 +10
==========================================
+ Hits 24288 24375 +87
+ Misses 14376 14276 -100
- Partials 1605 1628 +23
Continue to review full report at Codecov.
|
vmalert will repeat service discovery calls if on init first attempts returned nothing. Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks! have few minor comments
…fiers discovery Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
* move metrics structs used in the main vmalert package to utils so it can be reused by other subpackages * move metrics to alertmanager struct because it should handle those * support metrics registering/unregistering on notifier config changes * change behaviour of `vmalert_alerts_send_errors_total` - it now accounts for failed alerts, not HTTP calls. Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
Such situation is expected when user runs only recording rules Signed-off-by: hagen1778 <roman@victoriametrics.com>
Signed-off-by: hagen1778 <roman@victoriametrics.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
vmalert: support configuration file for notifiers * vmalert notifiers now can be configured via file see https://docs.victoriametrics.com/vmalert.html#notifier-configuration-file * add support of Consul service discovery for notifiers config see #1947 * add UI section for currently loaded/discovered notifiers * deprecate `-rule.configCheckInterval` in favour of `-configCheckInterval` * add ability to suppress logs for duplicated targets for notifiers discovery * change behaviour of `vmalert_alerts_send_errors_total` - it now accounts for failed alerts, not HTTP calls.
FYI, this pull request has been included into v1.73.0 release. |
see https://docs.victoriametrics.com/vmalert.html#notifier-configuration-file
see vmalert to support consul sd for discovering of notifier addresses #1947
-rule.configCheckInterval
in favour of-configCheckInterval
Signed-off-by: hagen1778 roman@victoriametrics.com