Skip to content

Commit 1fac1c7

Browse files
author
Julien Pivotto
authored
Merge pull request prometheus#8504 from rbauduin/require_alertname
promtool: alert_rule_test items require alertname
2 parents 2d172d0 + a7d64ca commit 1fac1c7

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

cmd/promtool/unittest.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,13 @@ func (tg *testGroup) test(evalInterval time.Duration, groupOrderMap map[string]i
188188
// Map of all the unit tests for given eval_time.
189189
alertTests := make(map[model.Duration][]alertTestCase)
190190
for _, alert := range tg.AlertRuleTests {
191+
if alert.Alertname == "" {
192+
var testGroupLog string
193+
if tg.TestGroupName != "" {
194+
testGroupLog = fmt.Sprintf(" (in TestGroup %s)", tg.TestGroupName)
195+
}
196+
return []error{errors.Errorf("an item under alert_rule_test misses required attribute alertname at eval_time %v%s", alert.EvalTime, testGroupLog)}
197+
}
191198
alertEvalTimesMap[alert.EvalTime] = struct{}{}
192199

193200
if _, ok := alertsInTest[alert.EvalTime]; !ok {

0 commit comments

Comments
 (0)