Revert 6155cc25ddaad253abcced4fcd8108d1be98e9c1

[codeboten]

This reverts commit 6155cc2: Make the option WithErrorUnused enabled by default when unmarshaling configuration (#9154)".

I propose reverting this change as it's blocking the release. It's causing problems and i'm not sure if there's a way to fix these tests without further changes in core as they call UnmarshalConfig which doesn't provide a way to pass in WithIgnoreUnused

opentelemetry-collector/component/config.go

Lines 30 to 36 in 6a726c9

	func UnmarshalConfig(conf *confmap.Conf, intoCfg Config) error {
	if cu, ok := intoCfg.(confmap.Unmarshaler); ok {
	return cu.Unmarshal(conf)
	}
	return conf.Unmarshal(intoCfg)
	}

See
https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/7466429082/job/20317807463?pr=30167

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (6a726c9) 90.76% compared to head (ac1747c) 90.76%.

Files	Patch %	Lines
component/config.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9251   +/-   ##
=======================================
  Coverage   90.76%   90.76%           
=======================================
  Files         341      341           
  Lines       18344    18340    -4     
=======================================
- Hits        16650    16647    -3     
+ Misses       1358     1356    -2     
- Partials      336      337    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.