Skip to content

Commit 7fd7054

Browse files
authored
[chore] Remove flaky TestConfigString test in datasetexporter (open-telemetry#37251)
#### Description This test will break `contrib-test` in opentelemetry-collector repo when we add new features in exporter spec. Removing this test as it seems not to be useful(only test string representation) and makes `contrib-test` flaky. Link: https://github.com/open-telemetry/opentelemetry-collector/actions/runs/12786594498/job/35643968597 cc @bogdandrutu #### Link to tracking issue Relevant to open-telemetry/opentelemetry-collector#12090
1 parent 52d1876 commit 7fd7054

File tree

1 file changed

+0
-45
lines changed

1 file changed

+0
-45
lines changed

exporter/datasetexporter/config_test.go

-45
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@ import (
99
"time"
1010

1111
"github.com/stretchr/testify/assert"
12-
"go.opentelemetry.io/collector/config/configretry"
1312
"go.opentelemetry.io/collector/confmap"
14-
"go.opentelemetry.io/collector/exporter/exporterhelper"
1513
)
1614

1715
func TestConfigUnmarshalUnknownAttributes(t *testing.T) {
@@ -102,49 +100,6 @@ func TestConfigValidate(t *testing.T) {
102100
}
103101
}
104102

105-
func TestConfigString(t *testing.T) {
106-
config := Config{
107-
DatasetURL: "https://example.com",
108-
APIKey: "secret",
109-
Debug: true,
110-
BufferSettings: BufferSettings{
111-
MaxLifetime: 123,
112-
PurgeOlderThan: 567,
113-
GroupBy: []string{"field1", "field2"},
114-
},
115-
TracesSettings: TracesSettings{
116-
exportSettings: exportSettings{
117-
ExportSeparator: "TTT",
118-
ExportDistinguishingSuffix: "UUU",
119-
},
120-
},
121-
LogsSettings: LogsSettings{
122-
ExportResourceInfo: true,
123-
ExportResourcePrefix: "AAA",
124-
ExportScopeInfo: true,
125-
ExportScopePrefix: "BBB",
126-
DecomposeComplexMessageField: true,
127-
DecomposedComplexMessagePrefix: "EEE",
128-
exportSettings: exportSettings{
129-
ExportSeparator: "CCC",
130-
ExportDistinguishingSuffix: "DDD",
131-
},
132-
},
133-
ServerHostSettings: ServerHostSettings{
134-
ServerHost: "foo-bar",
135-
UseHostName: false,
136-
},
137-
BackOffConfig: configretry.NewDefaultBackOffConfig(),
138-
QueueSettings: exporterhelper.NewDefaultQueueConfig(),
139-
TimeoutSettings: exporterhelper.NewDefaultTimeoutConfig(),
140-
}
141-
142-
assert.Equal(t,
143-
"DatasetURL: https://example.com; APIKey: [REDACTED] (6); Debug: true; BufferSettings: {MaxLifetime:123ns PurgeOlderThan:567ns GroupBy:[field1 field2] RetryInitialInterval:0s RetryMaxInterval:0s RetryMaxElapsedTime:0s RetryShutdownTimeout:0s MaxParallelOutgoing:0}; LogsSettings: {ExportResourceInfo:true ExportResourcePrefix:AAA ExportScopeInfo:true ExportScopePrefix:BBB DecomposeComplexMessageField:true DecomposedComplexMessagePrefix:EEE exportSettings:{ExportSeparator:CCC ExportDistinguishingSuffix:DDD}}; TracesSettings: {exportSettings:{ExportSeparator:TTT ExportDistinguishingSuffix:UUU}}; ServerHostSettings: {UseHostName:false ServerHost:foo-bar}; BackOffConfig: {Enabled:true InitialInterval:5s RandomizationFactor:0.5 Multiplier:1.5 MaxInterval:30s MaxElapsedTime:5m0s}; QueueSettings: {Enabled:true NumConsumers:10 QueueSize:1000 StorageID:<nil>}; TimeoutSettings: {Timeout:5s}",
144-
config.String(),
145-
)
146-
}
147-
148103
func TestConfigUseProvidedExportResourceInfoValue(t *testing.T) {
149104
f := NewFactory()
150105
config := f.CreateDefaultConfig().(*Config)

0 commit comments

Comments
 (0)