feat(ns-plug): add loglevel configuration to alert-proxy#1745

Merged

gsanchietti merged 9 commits into

mainfrom

Jun 26, 2026

gsanchietti commented Jun 24, 2026 •

edited

Loading

Member

Changes:

move alert rules from victoria-metrics to ns-plug
remove encrypted backup alert and custom evaluation interval
fix root filesystem label in Telegraf
return only firing alerts from ns.telegraf API
scrape vmalert self-metrics via -promscrape.config in victoria-metrics
add loglevel config to alert-proxy
relax alert rules to avoid multiple notifications and lower the number of flapping alerts

gsanchietti force-pushed the metrics_fixes branch 2 times, most recently from 525d512 to f0d3b90 Compare

June 25, 2026 14:04

gsanchietti commented

View reviewed changes

packages/ns-plug/files/config Outdated

gsanchietti commented

View reviewed changes

packages/ns-plug/files/ns-plug-alert-proxy Outdated

gsanchietti commented

View reviewed changes

packages/ns-plug/Makefile

gsanchietti commented

View reviewed changes

packages/victoria-metrics/files/vmalert-rules/backup.yaml

gsanchietti commented

View reviewed changes

packages/victoria-metrics/files/victoria-metrics.initd Outdated

gsanchietti commented

View reviewed changes

packages/victoria-metrics/README.md Outdated

gsanchietti marked this pull request as draft

June 25, 2026 14:39

gsanchietti and others added 4 commits

June 26, 2026 08:36


          feat(ns-plug): add loglevel config to alert-proxy

f56adc9

Default loglevel is 'warning' so only send failures appear in syslog.
Set loglevel=debug to restore verbose output (sent/dropped alerts).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>


          fix(ns-api): return only firing alerts

407242e

Avoid to should pending alerts inside the UI: pending
alerts could be confusing for users


          fix(alert): remove custom evaluation interval

03929d7

Use default evaluation interval (1m)


          fix(alert): remove encrypted backup alert

2c884da

Avoid sending a not so usufull alert.

gsanchietti force-pushed the metrics_fixes branch from f0d3b90 to 8970b4a Compare

June 26, 2026 06:36

gsanchietti added 4 commits

June 26, 2026 08:40


          chore: move alert rules to ns-plug package

1aa05e7

In case of package upgrade, ns-plug will require much less
space inside the overlay filesystem
Also increase check duration to avoid flapping.


          fix(metrics): change root filesystem label

06859c8

Make sure metrics and alerts are raised with path=/ label


          doc: improve custom rules info

36edbed


          fix(alert-proxy): remove config.disable_my_alerts

e620744

The option was added during a transition phase,
it's not required now.

gsanchietti force-pushed the metrics_fixes branch from 8970b4a to e620744 Compare

June 26, 2026 06:57


          doc(alerts): describe forwarding behavior

1a00be2

gsanchietti marked this pull request as ready for review

June 26, 2026 08:23

gsanchietti requested a review from Tbaile

June 26, 2026 08:23

Tbaile approved these changes

View reviewed changes

gsanchietti merged commit 35c2960 into main

3 checks passed

gsanchietti deleted the metrics_fixes branch

June 26, 2026 09:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet