open-telemetry / opentelemetry-collector

OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
4.32k stars 1.43k forks source link

Update go-viper to v2 causes issues #10387

Open codeboten opened 3 months ago

codeboten commented 3 months ago

Spent some time on this one. There's some messages that end up reporting an additional empty error:

expected: "decoding failed due to the following error(s):\n\nerror decoding 'err': never works"
actual  : "decoding failed due to the following error(s):\n\nerror decoding '': decoding failed due to the following error(s):\n\nerror decoding 'err': never works"

It's unclear to me why

Originally posted by @codeboten in https://github.com/open-telemetry/opentelemetry-collector/issues/10300#issuecomment-2160771395

codeboten commented 3 months ago

See work in progress here: https://github.com/open-telemetry/opentelemetry-collector/pull/10388