influxdata / influxdata/influxdb

Combine monitor.check and monitor.notify into a single function

Open
#16,038 0 comments 0 reactions 0 assignees View on GitHub
monitoring-and-alerting
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

Currently we use a combination of `monitor.check` and `monitor.notify` to create an alert using checks and notification rules. This is confusing for the base case user that wants to be immediately notified when an issue happens. We should create a flux function called `monitor.alert` that is the combination of both `monitor.check` and `monitor.notify`.

Link https://github.com/influxdata/influxdb/issues/15839

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the existing monitor.check and monitor.notify functions and the linked issue 15839 to understand their current behavior and scope. Define monitor.alert as the combined entry point, then verify that it supports the immediate-notification use case while preserving the existing checks and notification rules.

Written by the indexing model from the issue text.

Assessment

Domain
observability-sre
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.