vmalert: fail on absent notifiers

[AndrewChubatiuk]

fixes #1757

---

Summary by cubic

vmalert now returns an error if no notifier targets are found, instead of starting with an empty -notifier.url. This prevents silent misconfigurations; ExtraArgs notifier.url/notifier.config are respected. Fixes #1757.

• Bug Fixes

  • Return error when no notifiers are discovered via selectors or static config.
  • Do not append empty -notifier.url; use ExtraArgs notifier.url or notifier.config when provided.
  • Updated tests to reflect new behavior.

• Migration

  • Configure notifiers via spec.notifiers/spec.notifier, or set ExtraArgs: notifier.url or notifier.config.
  • Deployments that previously ran without notifiers will now fail until configured.

^{Written for commit fa4f447. Summary will update on new commits.}

[cubic-dev-ai]

1 issue found across 3 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="docs/CHANGELOG.md">

<violation number="1" location="docs/CHANGELOG.md:20">
P1: Rule violated: **Changelog Review Agent**

Changelog entries must describe the user-visible behavior before and after the change and avoid internal implementation details. This entry only states an internal action (“throw error”) without explaining prior behavior or the user-facing improvement, so it violates the required structure.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[vrutkovs]

This breaks e2e tests now

[AndrewChubatiuk]

fixed tests