Pablo Baeyens

@bogdandrutu This is waiting on your review

I would vote for stabilizing the `client` module before we decide on this

@Sanket-0510 Apologies, I likely won't have time to review until the end of the week

> ``` [...] > Best Practice: > For clarity and safety, consider explicitly mentioning the IP (127.0.0.1) instead of relying solely on 'localhost,' especially in sensitive setups. Keep things consistent...

> the empty value is still in use Where do we use the empty value?

I think open-telemetry/opentelemetry-collector-contrib/pull/31263 should fix the remaining issues with contrib tests

cc @open-telemetry/collector-approvers this is ready to review, but I won't merge this for v0.96.0, will wait until after the release

I am still not sure why 4 & 5 need to go before 6. It may be the case for specific config modules, but certainly not for all of them,...

I think we should decide on #9532 first, since if we do it the proposed change here wouldn't work. It's still valuable to think about the desired behavior here without...

For testing you can replace the logger with https://pkg.go.dev/go.uber.org/zap/zaptest/observer