|
|
|
// Copyright 2013 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.
|
|
|
|
|
|
|
|
package rules
|
|
|
|
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"testing"
|
|
|
|
"time"
|
|
|
|
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
|
|
|
|
"github.com/prometheus/prometheus/model/labels"
|
|
|
|
"github.com/prometheus/prometheus/model/timestamp"
|
|
|
|
"github.com/prometheus/prometheus/promql"
|
|
|
|
"github.com/prometheus/prometheus/promql/parser"
|
|
|
|
"github.com/prometheus/prometheus/util/teststorage"
|
|
|
|
)
|
|
|
|
|
|
|
|
func TestRuleEval(t *testing.T) {
|
|
|
|
storage := teststorage.New(t)
|
|
|
|
defer storage.Close()
|
|
|
|
|
|
|
|
opts := promql.EngineOpts{
|
|
|
|
Logger: nil,
|
|
|
|
Reg: nil,
|
|
|
|
MaxSamples: 10,
|
|
|
|
Timeout: 10 * time.Second,
|
|
|
|
}
|
|
|
|
|
|
|
|
engine := promql.NewEngine(opts)
|
|
|
|
ctx, cancelCtx := context.WithCancel(context.Background())
|
|
|
|
defer cancelCtx()
|
promql: Allow per-query contexts.
For Weaveworks' Frankenstein, we need to support multitenancy. In
Frankenstein, we initially solved this without modifying the promql
package at all: we constructed a new promql.Engine for every
query and injected a storage implementation into that engine which would
be primed to only collect data for a given user.
This is problematic to upstream, however. Prometheus assumes that there
is only one engine: the query concurrency gate is part of the engine,
and the engine contains one central cancellable context to shut down all
queries. Also, creating a new engine for every query seems like overkill.
Thus, we want to be able to pass per-query contexts into a single engine.
This change gets rid of the promql.Engine's built-in base context and
allows passing in a per-query context instead. Central cancellation of
all queries is still possible by deriving all passed-in contexts from
one central one, but this is now the responsibility of the caller. The
central query context is now created in main() and passed into the
relevant components (web handler / API, rule manager).
In a next step, the per-query context would have to be passed to the
storage implementation, so that the storage can implement multi-tenancy
or other features based on the contextual information.
8 years ago
|
|
|
|
|
|
|
now := time.Now()
|
|
|
|
|
|
|
|
suite := []struct {
|
|
|
|
name string
|
|
|
|
expr parser.Expr
|
|
|
|
labels labels.Labels
|
|
|
|
result promql.Vector
|
|
|
|
err string
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
name: "nolabels",
|
|
|
|
expr: &parser.NumberLiteral{Val: 1},
|
|
|
|
labels: labels.Labels{},
|
|
|
|
result: promql.Vector{promql.Sample{
|
|
|
|
Metric: labels.FromStrings("__name__", "nolabels"),
|
|
|
|
Point: promql.Point{V: 1, T: timestamp.FromTime(now)},
|
|
|
|
}},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "labels",
|
|
|
|
expr: &parser.NumberLiteral{Val: 1},
|
|
|
|
labels: labels.FromStrings("foo", "bar"),
|
|
|
|
result: promql.Vector{promql.Sample{
|
|
|
|
Metric: labels.FromStrings("__name__", "labels", "foo", "bar"),
|
|
|
|
Point: promql.Point{V: 1, T: timestamp.FromTime(now)},
|
|
|
|
}},
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
for _, test := range suite {
|
|
|
|
rule := NewRecordingRule(test.name, test.expr, test.labels)
|
|
|
|
result, err := rule.Eval(ctx, now, EngineQueryFunc(engine, storage), nil, 0)
|
|
|
|
if test.err == "" {
|
|
|
|
require.NoError(t, err)
|
|
|
|
} else {
|
|
|
|
require.Equal(t, test.err, err.Error())
|
|
|
|
}
|
|
|
|
require.Equal(t, test.result, result)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// TestRuleEvalDuplicate tests for duplicate labels in recorded metrics, see #5529.
|
|
|
|
func TestRuleEvalDuplicate(t *testing.T) {
|
|
|
|
storage := teststorage.New(t)
|
|
|
|
defer storage.Close()
|
|
|
|
|
|
|
|
opts := promql.EngineOpts{
|
|
|
|
Logger: nil,
|
|
|
|
Reg: nil,
|
|
|
|
MaxSamples: 10,
|
|
|
|
Timeout: 10 * time.Second,
|
|
|
|
}
|
|
|
|
|
|
|
|
engine := promql.NewEngine(opts)
|
|
|
|
ctx, cancelCtx := context.WithCancel(context.Background())
|
|
|
|
defer cancelCtx()
|
|
|
|
|
|
|
|
now := time.Now()
|
|
|
|
|
|
|
|
expr, _ := parser.ParseExpr(`vector(0) or label_replace(vector(0),"test","x","","")`)
|
|
|
|
rule := NewRecordingRule("foo", expr, labels.FromStrings("test", "test"))
|
|
|
|
_, err := rule.Eval(ctx, now, EngineQueryFunc(engine, storage), nil, 0)
|
|
|
|
require.Error(t, err)
|
|
|
|
require.EqualError(t, err, "vector contains metrics with the same labelset after applying rule labels")
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestRecordingRuleLimit(t *testing.T) {
|
|
|
|
suite, err := promql.NewTest(t, `
|
|
|
|
load 1m
|
|
|
|
metric{label="1"} 1
|
|
|
|
metric{label="2"} 1
|
|
|
|
`)
|
|
|
|
require.NoError(t, err)
|
|
|
|
defer suite.Close()
|
|
|
|
|
|
|
|
require.NoError(t, suite.Run())
|
|
|
|
|
|
|
|
tests := []struct {
|
|
|
|
limit int
|
|
|
|
err string
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
limit: 0,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
limit: -1,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
limit: 2,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
limit: 1,
|
|
|
|
err: "exceeded limit of 1 with 2 series",
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
expr, _ := parser.ParseExpr(`metric > 0`)
|
|
|
|
rule := NewRecordingRule(
|
|
|
|
"foo",
|
|
|
|
expr,
|
|
|
|
labels.FromStrings("test", "test"),
|
|
|
|
)
|
|
|
|
|
|
|
|
evalTime := time.Unix(0, 0)
|
|
|
|
|
|
|
|
for _, test := range tests {
|
|
|
|
_, err := rule.Eval(suite.Context(), evalTime, EngineQueryFunc(suite.QueryEngine(), suite.Storage()), nil, test.limit)
|
|
|
|
if err != nil {
|
|
|
|
require.EqualError(t, err, test.err)
|
|
|
|
} else if test.err != "" {
|
|
|
|
t.Errorf("Expected error %s, got none", test.err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// TestRecordingEvalWithOrigin checks that the recording rule details are passed through the context.
|
|
|
|
func TestRecordingEvalWithOrigin(t *testing.T) {
|
|
|
|
ctx := context.Background()
|
|
|
|
now := time.Now()
|
|
|
|
|
|
|
|
const (
|
|
|
|
name = "my-recording-rule"
|
|
|
|
query = `count(metric{foo="bar"})`
|
|
|
|
)
|
|
|
|
|
|
|
|
var (
|
|
|
|
detail RuleDetail
|
|
|
|
lbs = labels.FromStrings("foo", "bar")
|
|
|
|
)
|
|
|
|
|
|
|
|
expr, err := parser.ParseExpr(query)
|
|
|
|
require.NoError(t, err)
|
|
|
|
|
|
|
|
rule := NewRecordingRule(name, expr, lbs)
|
|
|
|
_, err = rule.Eval(ctx, now, func(ctx context.Context, qs string, _ time.Time) (promql.Vector, error) {
|
|
|
|
detail = FromOriginContext(ctx)
|
|
|
|
return nil, nil
|
|
|
|
}, nil, 0)
|
|
|
|
|
|
|
|
require.NoError(t, err)
|
|
|
|
require.Equal(t, detail, NewRuleDetail(rule))
|
|
|
|
}
|