Skip to content

Commit

Permalink
Add label matchers parser
Browse files Browse the repository at this point in the history
This commit adds the new label matchers parser as proposed in #3353.
Included is a number of compliance tests comparing the grammar
supported in the new parser with the existing parser in pkg/labels.
The compliance tests can be run passing the "compliance" tag when
running go test.

Signed-off-by: George Robinson <george.robinson@grafana.com>
  • Loading branch information
grobinson-grafana committed Aug 9, 2023
1 parent 7411e54 commit 24b7b52
Show file tree
Hide file tree
Showing 6 changed files with 1,818 additions and 0 deletions.
376 changes: 376 additions & 0 deletions matchers/compliance/compliance_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,376 @@
// Copyright 2023 The Prometheus Authors
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

//go:build compliance

package compliance

import (
"reflect"
"testing"

"github.com/prometheus/alertmanager/matchers/parse"
"github.com/prometheus/alertmanager/pkg/labels"
)

func TestCompliance(t *testing.T) {
for _, tc := range []struct {
input string
want labels.Matchers
err string
}{
{
input: `{}`,
want: make(labels.Matchers, 0),
},
{
input: `{foo='}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "'")
return append(ms, m)
}(),
},
{
input: "{foo=`}",
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "`")
return append(ms, m)
}(),
},
{
input: "{foo=\\\"}",
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "\"")
return append(ms, m)
}(),
},
{
input: `{foo=bar}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
return append(ms, m)
}(),
},
{
input: `{foo="bar"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
return append(ms, m)
}(),
},
{
input: `{foo=~bar.*}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchRegexp, "foo", "bar.*")
return append(ms, m)
}(),
},
{
input: `{foo=~"bar.*"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchRegexp, "foo", "bar.*")
return append(ms, m)
}(),
},
{
input: `{foo!=bar}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchNotEqual, "foo", "bar")
return append(ms, m)
}(),
},
{
input: `{foo!="bar"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchNotEqual, "foo", "bar")
return append(ms, m)
}(),
},
{
input: `{foo!~bar.*}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchNotRegexp, "foo", "bar.*")
return append(ms, m)
}(),
},
{
input: `{foo!~"bar.*"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchNotRegexp, "foo", "bar.*")
return append(ms, m)
}(),
},
{
input: `{foo="bar", baz!="quux"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotEqual, "baz", "quux")
return append(ms, m, m2)
}(),
},
{
input: `{foo="bar", baz!~"quux.*"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotRegexp, "baz", "quux.*")
return append(ms, m, m2)
}(),
},
{
input: `{foo="bar",baz!~".*quux", derp="wat"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotRegexp, "baz", ".*quux")
m3, _ := labels.NewMatcher(labels.MatchEqual, "derp", "wat")
return append(ms, m, m2, m3)
}(),
},
{
input: `{foo="bar", baz!="quux", derp="wat"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotEqual, "baz", "quux")
m3, _ := labels.NewMatcher(labels.MatchEqual, "derp", "wat")
return append(ms, m, m2, m3)
}(),
},
{
input: `{foo="bar", baz!~".*quux.*", derp="wat"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotRegexp, "baz", ".*quux.*")
m3, _ := labels.NewMatcher(labels.MatchEqual, "derp", "wat")
return append(ms, m, m2, m3)
}(),
},
{
input: `{foo="bar", instance=~"some-api.*"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchRegexp, "instance", "some-api.*")
return append(ms, m, m2)
}(),
},
{
input: `{foo=""}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "")
return append(ms, m)
}(),
},
{
input: `{foo="bar,quux", job="job1"}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar,quux")
m2, _ := labels.NewMatcher(labels.MatchEqual, "job", "job1")
return append(ms, m, m2)
}(),
},
{
input: `{foo = "bar", dings != "bums", }`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotEqual, "dings", "bums")
return append(ms, m, m2)
}(),
},
{
input: `foo=bar,dings!=bums`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar")
m2, _ := labels.NewMatcher(labels.MatchNotEqual, "dings", "bums")
return append(ms, m, m2)
}(),
},
{
input: `{quote="She said: \"Hi, ladies! That's gender-neutral…\""}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "quote", `She said: "Hi, ladies! That's gender-neutral…"`)
return append(ms, m)
}(),
},
{
input: `statuscode=~"5.."`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchRegexp, "statuscode", "5..")
return append(ms, m)
}(),
},
{
input: `tricky=~~~`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchRegexp, "tricky", "~~")
return append(ms, m)
}(),
},
{
input: `trickier==\\=\=\"`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "trickier", `=\=\="`)
return append(ms, m)
}(),
},
{
input: `contains_quote != "\"" , contains_comma !~ "foo,bar" , `,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchNotEqual, "contains_quote", `"`)
m2, _ := labels.NewMatcher(labels.MatchNotRegexp, "contains_comma", "foo,bar")
return append(ms, m, m2)
}(),
},
{
input: `{foo=bar}}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar}")
return append(ms, m)
}(),
},
{
input: `{foo=bar}},}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "bar}}")
return append(ms, m)
}(),
},
{
input: `{foo=,bar=}}`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m1, _ := labels.NewMatcher(labels.MatchEqual, "foo", "")
m2, _ := labels.NewMatcher(labels.MatchEqual, "bar", "}")
return append(ms, m1, m2)
}(),
},
{
input: `job=`,
want: func() labels.Matchers {
m, _ := labels.NewMatcher(labels.MatchEqual, "job", "")
return labels.Matchers{m}
}(),
},
{
input: `{,}`,
err: "bad matcher format: ",
},
{
input: `job="value`,
err: `matcher value contains unescaped double quote: "value`,
},
{
input: `job=value"`,
err: `matcher value contains unescaped double quote: value"`,
},
{
input: `trickier==\\=\=\""`,
err: `matcher value contains unescaped double quote: =\\=\=\""`,
},
{
input: `contains_unescaped_quote = foo"bar`,
err: `matcher value contains unescaped double quote: foo"bar`,
},
{
input: `{invalid-name = "valid label"}`,
err: `bad matcher format: invalid-name = "valid label"`,
},
{
input: `{foo=~"invalid[regexp"}`,
err: "error parsing regexp: missing closing ]: `[regexp)$`",
},
// Double escaped strings.
{
input: `"{foo=\"bar"}`,
err: `bad matcher format: "{foo=\"bar"`,
},
{
input: `"foo=\"bar"`,
err: `bad matcher format: "foo=\"bar"`,
},
{
input: `"foo=\"bar\""`,
err: `bad matcher format: "foo=\"bar\""`,
},
{
input: `"foo=\"bar\"`,
err: `bad matcher format: "foo=\"bar\"`,
},
{
input: `"{foo=\"bar\"}"`,
err: `bad matcher format: "{foo=\"bar\"}"`,
},
{
input: `"foo="bar""`,
err: `bad matcher format: "foo="bar""`,
},
{
input: `{{foo=`,
err: `bad matcher format: {foo=`,
},
{
input: `{foo=`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "")
return append(ms, m)
}(),
},
{
input: `{foo=}b`,
want: func() labels.Matchers {
ms := labels.Matchers{}
m, _ := labels.NewMatcher(labels.MatchEqual, "foo", "}b")
return append(ms, m)
}(),
},
} {
t.Run(tc.input, func(t *testing.T) {
got, err := parse.Parse(tc.input)
if err != nil && tc.err == "" {
t.Fatalf("got error where none expected: %v", err)
}
if err == nil && tc.err != "" {
t.Fatalf("expected error but got none: %v", tc.err)
}
if !reflect.DeepEqual(got, tc.want) {
t.Fatalf("labels not equal:\ngot %v\nwant %v", got, tc.want)
}
})
}
}
Loading

0 comments on commit 24b7b52

Please sign in to comment.