open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
3.1k stars 2.39k forks source link

[SQL Server] expose server.port #35183

Open atoulme opened 2 months ago

atoulme commented 2 months ago

Component(s)

receiver/sqlserver

Is your feature request related to a problem? Please describe.

Similar to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32810 we find that multiple SQL server instances running on the same host will be commingled.

Describe the solution you'd like

the SQL server receiver should expose a server.port attribute similar to what the mongodbreceiver is doing.

Describe alternatives you've considered

No response

Additional context

No response

github-actions[bot] commented 2 months ago

Pinging code owners:

atoulme commented 2 months ago

The metadata.yaml would be enhanced with

  server.address:
    description: The address of the SQL server host.
    enabled: true
    type: string
  server.port:
    description: The port of the SQL server host.
    enabled: false
    type: int