Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

db.mssql.instance_name vs db.instance.id and server.port #727

Closed
lmolkova opened this issue Feb 9, 2024 · 3 comments
Closed

db.mssql.instance_name vs db.instance.id and server.port #727

lmolkova opened this issue Feb 9, 2024 · 3 comments
Assignees

Comments

@lmolkova
Copy link
Contributor

lmolkova commented Feb 9, 2024

Attribute Type Description Examples
db.mssql.instance_name string The Microsoft SQL Server instance name connecting to. This name is used to determine the port of a named instance. [1] MSSQLSERVER

[1]: If setting a db.mssql.instance_name, server.port is no longer required (but still recommended if non-standard).

Issues:

  1. It sounds like generic db.instance.id can be used. Related Should db.instance.id replace elastic and mssql specific attributes? #725
  2. Based on SQL Server docs, ports are dynamic, so server.port is not known at all and connection is made against [hostname + instance name](Based on named instances docs). So probably network.address.port should be use here instead. Related: DB/messaging/rpc(?): should logical operations include network.* attributes #690
@Oberon00
Copy link
Member

Oberon00 commented Feb 9, 2024

See #725 (comment):

AFAIK, the MS SQL instance does not match your proposed description of db.instance.id. It seems to be a quite special concept, I wonder if you will find anything equivalent in any other db.
MS SQL instances are not members of a cluster. Depending on which "instance" you connect to, you will have different databases. This instance concept seems to allow running logically distinct DBs on the same hostname, so it's rather the opposite of clustering.

https://learn.microsoft.com/en-us/sql/database-engine/configure-windows/database-engine-instances-sql-server?view=sql-server-ver16

@trask
Copy link
Member

trask commented Apr 24, 2024

@lmolkova is this resolved by #911?

@lmolkova
Copy link
Contributor Author

@lmolkova is this resolved by #911?

yes, closing this one.

The tl;dr: db.mssql.instance_name is now part of db.namespace following {instance_name.database name} pattern. Default instance name is not populated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

4 participants