RepoMirrors/alertmanager
1
0
Fork 0
mirror of https://github.com/prometheus/alertmanager synced 2024-12-25 23:52:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

The default group_by is meant to be no labels. (#1287)

Browse Source 
This is what the intended default is, and what
the documentation says.
This commit is contained in:
Brian Brazil 2018-03-16 17:39:23 +00:00 committed by stuart nelson
parent 8c9e0cf50c
commit aa950668bf
2 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
dispatch/route.go
View File

@ -31,9 +31,7 @@ var DefaultRouteOpts = RouteOpts{
GroupWait: 30 * time.Second,
GroupInterval: 5 * time.Minute,
RepeatInterval: 4 * time.Hour,
GroupBy: map[model.LabelName]struct{}{
model.AlertNameLabel: struct{}{},
},
GroupBy: map[model.LabelName]struct{}{},
}
// A Route is a node that contains definitions of how to handle alerts.

6
test/acceptance/send_test.go
View File

@ -33,7 +33,7 @@ func TestMergeAlerts(t *testing.T) {
conf := `
route:
receiver: "default"
group_by: []
group_by: [alertname]
group_wait: 1s
group_interval: 1s
repeat_interval: 1ms
@ -109,7 +109,7 @@ func TestRepeat(t *testing.T) {
conf := `
route:
receiver: "default"
group_by: []
group_by: [alertname]
group_wait: 1s
group_interval: 1s
repeat_interval: 1ms
@ -168,7 +168,7 @@ func TestRetry(t *testing.T) {
conf := `
route:
receiver: "default"
group_by: []
group_by: [alertname]
group_wait: 1s
group_interval: 1s
repeat_interval: 3s