mirror of
https://github.com/prometheus/prometheus.git
synced 2024-11-15 10:04:07 -08:00
3326df42bb
* Removing global state modification on unit tests (fix #10033 #10034) The config.DefaultRemoteReadConfig and config.DefaultRemoteWriteConfig instances hold global state. Unit tests were changing their url.URL reference globally causing false positives when tests were ran through package. Two helper functions were created to copy those global values instead of changing them in place to fix null point when running unit tests by method instead of by package. Signed-off-by: Leonardo Zamariola <leonardo.zamariola@gmail.com> * Fixing pull request suggestions Copying by value from default config Signed-off-by: Leonardo Zamariola <leonardo.zamariola@gmail.com>
150 lines
4.1 KiB
Go
150 lines
4.1 KiB
Go
// Copyright 2019 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 remote
|
|
|
|
import (
|
|
"net/url"
|
|
"testing"
|
|
|
|
common_config "github.com/prometheus/common/config"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/prometheus/prometheus/config"
|
|
"github.com/prometheus/prometheus/model/labels"
|
|
)
|
|
|
|
func TestStorageLifecycle(t *testing.T) {
|
|
dir := t.TempDir()
|
|
|
|
s := NewStorage(nil, nil, nil, dir, defaultFlushDeadline, nil)
|
|
conf := &config.Config{
|
|
GlobalConfig: config.DefaultGlobalConfig,
|
|
RemoteWriteConfigs: []*config.RemoteWriteConfig{
|
|
// We need to set URL's so that metric creation doesn't panic.
|
|
baseRemoteWriteConfig("http://test-storage.com"),
|
|
},
|
|
RemoteReadConfigs: []*config.RemoteReadConfig{
|
|
baseRemoteReadConfig("http://test-storage.com"),
|
|
},
|
|
}
|
|
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
|
|
// make sure remote write has a queue.
|
|
require.Equal(t, 1, len(s.rws.queues))
|
|
|
|
// make sure remote write has a queue.
|
|
require.Equal(t, 1, len(s.queryables))
|
|
|
|
err := s.Close()
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func TestUpdateRemoteReadConfigs(t *testing.T) {
|
|
dir := t.TempDir()
|
|
|
|
s := NewStorage(nil, nil, nil, dir, defaultFlushDeadline, nil)
|
|
|
|
conf := &config.Config{
|
|
GlobalConfig: config.GlobalConfig{},
|
|
}
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 0, len(s.queryables))
|
|
|
|
conf.RemoteReadConfigs = []*config.RemoteReadConfig{
|
|
baseRemoteReadConfig("http://test-storage.com"),
|
|
}
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 1, len(s.queryables))
|
|
|
|
err := s.Close()
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func TestFilterExternalLabels(t *testing.T) {
|
|
dir := t.TempDir()
|
|
|
|
s := NewStorage(nil, nil, nil, dir, defaultFlushDeadline, nil)
|
|
|
|
conf := &config.Config{
|
|
GlobalConfig: config.GlobalConfig{
|
|
ExternalLabels: labels.Labels{labels.Label{Name: "foo", Value: "bar"}},
|
|
},
|
|
}
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 0, len(s.queryables))
|
|
|
|
conf.RemoteReadConfigs = []*config.RemoteReadConfig{
|
|
baseRemoteReadConfig("http://test-storage.com"),
|
|
}
|
|
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 1, len(s.queryables))
|
|
require.Equal(t, 1, len(s.queryables[0].(*sampleAndChunkQueryableClient).externalLabels))
|
|
|
|
err := s.Close()
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func TestIgnoreExternalLabels(t *testing.T) {
|
|
dir := t.TempDir()
|
|
|
|
s := NewStorage(nil, nil, nil, dir, defaultFlushDeadline, nil)
|
|
|
|
conf := &config.Config{
|
|
GlobalConfig: config.GlobalConfig{
|
|
ExternalLabels: labels.Labels{labels.Label{Name: "foo", Value: "bar"}},
|
|
},
|
|
}
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 0, len(s.queryables))
|
|
|
|
conf.RemoteReadConfigs = []*config.RemoteReadConfig{
|
|
baseRemoteReadConfig("http://test-storage.com"),
|
|
}
|
|
|
|
conf.RemoteReadConfigs[0].FilterExternalLabels = false
|
|
|
|
require.NoError(t, s.ApplyConfig(conf))
|
|
require.Equal(t, 1, len(s.queryables))
|
|
require.Equal(t, 0, len(s.queryables[0].(*sampleAndChunkQueryableClient).externalLabels))
|
|
|
|
err := s.Close()
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
// baseRemoteWriteConfig copy values from global Default Write config
|
|
// to avoid change global state and cross impact test execution
|
|
func baseRemoteWriteConfig(host string) *config.RemoteWriteConfig {
|
|
cfg := config.DefaultRemoteWriteConfig
|
|
cfg.URL = &common_config.URL{
|
|
URL: &url.URL{
|
|
Host: host,
|
|
},
|
|
}
|
|
return &cfg
|
|
}
|
|
|
|
// baseRemoteReadConfig copy values from global Default Read config
|
|
// to avoid change global state and cross impact test execution
|
|
func baseRemoteReadConfig(host string) *config.RemoteReadConfig {
|
|
cfg := config.DefaultRemoteReadConfig
|
|
cfg.URL = &common_config.URL{
|
|
URL: &url.URL{
|
|
Host: host,
|
|
},
|
|
}
|
|
return &cfg
|
|
}
|