mirror of
https://github.com/hanzoai/telemetry.git
synced 2026-10-30 20:01:49 +00:00
Dmitry AnoshinandGitHub
ae38f549da
Related to https://github.com/open-telemetry/opentelemetry-collector/issues/14208 This change allows component authors to register one optional deprecated alias per component type, enabling smooth migrations when renaming components. When an alias is used in configuration, the collector will automatically resolve it to the original component and log a deprecation warning. Changes: - Add `WithDeprecatedTypeAlias` factory option to experimental modules (xreceiver, xexporter, xprocessor, xconnector) - Generate alias expansion code in builder template that: - Populates factory maps with aliases pointing to original factories - Updates module information maps for aliased types - Validates that aliases don't conflict with existing component types - Add deprecation warnings that log when an alias is used: "Using deprecated alias <alias>; use <original> instead" Extensions are left aside for now.
82 lines
2.8 KiB
Go
82 lines
2.8 KiB
Go
// Copyright The OpenTelemetry Authors
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package xprocessor
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"go.opentelemetry.io/collector/component"
|
|
"go.opentelemetry.io/collector/consumer/consumertest"
|
|
"go.opentelemetry.io/collector/consumer/xconsumer"
|
|
"go.opentelemetry.io/collector/internal/componentalias"
|
|
"go.opentelemetry.io/collector/processor"
|
|
"go.opentelemetry.io/collector/processor/internal"
|
|
)
|
|
|
|
var testID = component.MustNewID("test")
|
|
|
|
func TestNewFactoryWithProfiles(t *testing.T) {
|
|
testType := component.MustNewType("test")
|
|
defaultCfg := struct{}{}
|
|
factory := NewFactory(
|
|
testType,
|
|
func() component.Config { return &defaultCfg },
|
|
WithProfiles(createProfiles, component.StabilityLevelAlpha),
|
|
)
|
|
assert.Equal(t, testType, factory.Type())
|
|
assert.EqualValues(t, &defaultCfg, factory.CreateDefaultConfig())
|
|
|
|
assert.Equal(t, component.StabilityLevelAlpha, factory.ProfilesStability())
|
|
_, err := factory.CreateProfiles(context.Background(), processor.Settings{ID: testID}, &defaultCfg, consumertest.NewNop())
|
|
require.NoError(t, err)
|
|
|
|
wrongID := component.MustNewID("wrong")
|
|
wrongIDErrStr := internal.ErrIDMismatch(wrongID, testType).Error()
|
|
_, err = factory.CreateProfiles(context.Background(), processor.Settings{ID: wrongID}, &defaultCfg, consumertest.NewNop())
|
|
assert.EqualError(t, err, wrongIDErrStr)
|
|
}
|
|
|
|
var nopInstance = &nopProcessor{
|
|
Consumer: consumertest.NewNop(),
|
|
}
|
|
|
|
// nopProcessor stores consumed traces and metrics for testing purposes.
|
|
type nopProcessor struct {
|
|
component.StartFunc
|
|
component.ShutdownFunc
|
|
consumertest.Consumer
|
|
}
|
|
|
|
func createProfiles(context.Context, processor.Settings, component.Config, xconsumer.Profiles) (Profiles, error) {
|
|
return nopInstance, nil
|
|
}
|
|
|
|
func TestNewFactoryWithDeprecatedAlias(t *testing.T) {
|
|
testType := component.MustNewType("newname")
|
|
aliasType := component.MustNewType("oldname")
|
|
defaultCfg := struct{}{}
|
|
|
|
f := NewFactory(
|
|
testType,
|
|
func() component.Config { return &defaultCfg },
|
|
WithProfiles(createProfiles, component.StabilityLevelAlpha),
|
|
WithDeprecatedTypeAlias(aliasType),
|
|
)
|
|
|
|
assert.Equal(t, testType, f.Type())
|
|
assert.Equal(t, aliasType, f.(*factory).Factory.(componentalias.TypeAliasHolder).DeprecatedAlias())
|
|
assert.EqualValues(t, &defaultCfg, f.CreateDefaultConfig())
|
|
|
|
_, err := f.CreateProfiles(context.Background(), processor.Settings{ID: component.MustNewID("newname")}, &defaultCfg, consumertest.NewNop())
|
|
require.NoError(t, err)
|
|
_, err = f.CreateProfiles(context.Background(), processor.Settings{ID: component.MustNewID("oldname")}, &defaultCfg, consumertest.NewNop())
|
|
require.NoError(t, err)
|
|
_, err = f.CreateProfiles(context.Background(), processor.Settings{ID: component.MustNewID("wrongname")}, &defaultCfg, consumertest.NewNop())
|
|
require.Error(t, err)
|
|
}
|