Merge pull request #69 from K-Phoen/notification-channels

Allow notification channels to be defined as a list in alerts
K-Phoen · Jun 16, 2020 · a7962c0 · a7962c0
2 parents e44987b + 382ea98
commit a7962c0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/decoder/graph.go b/decoder/graph.go
@@ -173,6 +173,7 @@ type GraphAlert struct {
 	For              string
 	If               []AlertCondition
 	Notify           string
+	Notifications    []string
 	Message          string
 	OnNoData         string `yaml:"on_no_data"`
 	OnExecutionError string `yaml:"on_execution_error"`
@@ -223,6 +224,10 @@ func (a GraphAlert) toOptions() ([]alert.Option, error) {
 		opts = append(opts, alert.Message(a.Message))
 	}
 
+	for _, channel := range a.Notifications {
+		opts = append(opts, alert.NotifyChannel(channel))
+	}
+
 	for _, condition := range a.If {
 		conditionOpt, err := condition.toOption()
 		if err != nil {

diff --git a/doc/graph_panels_yaml.md b/doc/graph_panels_yaml.md
@@ -39,8 +39,10 @@ rows:
           title: Too many heap allocations
           evaluate_every: 1m
           for: 1m
-          # ID of the notification channel
-          notify: 1
+          # UID of the notification channel
+          notify: "P-N3fxuZz"
+          # UIDs of the notification channels
+          #notifications: ["P-N3fxuZz"]
           message: "Wow, a we're allocating a lot."
           # Valid values are: no_data, alerting, keep_state, ok
           on_no_data: alerting