[receiver/sqlserver] Allow server config option to include port about opentelemetry-collector-contrib HOT 3 CLOSED

Fair point, and I agree. It does make the configuration a bit more confusing.

My original concern here was the extent of changes required in the observer extension, as I wasn't very familiar with its inner-workings. After more of a look though, it appears to be a pretty simple change to be able to expose the values we need in the extension, so I think I'll move my efforts there instead. 👍

from opentelemetry-collector-contrib.

Pinging code owners:

• receiver/sqlserver: @djaglowski @StefanKurek

See Adding Labels via Comments if you do not have permissions to add labels yourself.

from opentelemetry-collector-contrib.

IMO, this seems like a problem with the observer extension's behavior. The proposed to this receiver introduces ambiguity into the configuration. Are you sure there isn't a reasonable change we could make to the observer package instead?

from opentelemetry-collector-contrib.