mirror of
https://github.com/hanzoai/o11y.git
synced 2026-10-15 12:58:33 +00:00
zeekayandGitHub
b838309bd6
281d622 renamed the env provider prefix SIGNOZ_ -> O11Y_ and updated only
provider_test.go, leaving every downstream env-provider consumer test still
setting the old SIGNOZ_ prefix. Those envs are now ignored by the O11Y_ provider,
so 7 packages failed `go test ./...` (CI red on main): pprof, web, sqlstore,
alertmanager, instrumentation, apiserver, cmd.
Rename the leading env prefix to O11Y_ in these tests (and align alertmanager's
`const prefix` + clearEnv helper). Scope is strictly the env-config prefix:
ClickHouse wire identifiers (constants.SIGNOZ_*_TABLENAME/_DBNAME) and query
reserved vars (SIGNOZ_START_TIME/#SIGNOZ_VALUE) are the datastore schema/query
contract and are deliberately untouched, matching 281d622's "store stays
ClickHouse wire-protocol".
go build ./... green; go test -count=1 -skip '^TestEmailRejected$' ./... = 125
packages OK, 0 failures.
58 lines
1.2 KiB
Go
58 lines
1.2 KiB
Go
package apiserver
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/hanzoai/o11y/pkg/config"
|
|
"github.com/hanzoai/o11y/pkg/config/envprovider"
|
|
"github.com/hanzoai/o11y/pkg/factory"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestNewWithEnvProvider(t *testing.T) {
|
|
t.Setenv("O11Y_APISERVER_TIMEOUT_DEFAULT", "70s")
|
|
t.Setenv("O11Y_APISERVER_TIMEOUT_MAX", "700s")
|
|
t.Setenv("O11Y_APISERVER_TIMEOUT_EXCLUDED__ROUTES", "/excluded1,/excluded2")
|
|
t.Setenv("O11Y_APISERVER_LOGGING_EXCLUDED__ROUTES", "/api/v1/health1")
|
|
|
|
conf, err := config.New(
|
|
context.Background(),
|
|
config.ResolverConfig{
|
|
Uris: []string{"env:"},
|
|
ProviderFactories: []config.ProviderFactory{
|
|
envprovider.NewFactory(),
|
|
},
|
|
},
|
|
[]factory.ConfigFactory{
|
|
NewConfigFactory(),
|
|
},
|
|
)
|
|
require.NoError(t, err)
|
|
|
|
actual := &Config{}
|
|
err = conf.Unmarshal("apiserver", actual)
|
|
|
|
require.NoError(t, err)
|
|
|
|
expected := &Config{
|
|
Timeout: Timeout{
|
|
Default: 70 * time.Second,
|
|
Max: 700 * time.Second,
|
|
ExcludedRoutes: []string{
|
|
"/excluded1",
|
|
"/excluded2",
|
|
},
|
|
},
|
|
Logging: Logging{
|
|
ExcludedRoutes: []string{
|
|
"/api/v1/health1",
|
|
},
|
|
},
|
|
}
|
|
|
|
assert.Equal(t, expected, actual)
|
|
}
|