-
Notifications
You must be signed in to change notification settings - Fork 174
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
How to capture DB.USER #1142
Comments
cc @open-telemetry/semconv-db-approvers |
@jerry-shao thank you for reporting this! Could you please share some details:
Thanks! |
Discussed in June 19 SIG meeting. Initially I thought we might use We are thinking it likely makes sense to embed the I've added the topic to the next general semantic convention SIG meeting. Note: if we do bring it back as |
Related: #818 |
Also: #1172 It's common to access databases (in the cloud, in k8s with identity providers, etc) using non-human identities which can be called groups, roles, etc. Or with named/unnamed tokens. It'd be difficult to describe all possible kinds of identities using
It seems a new version of |
Discussed in DB semconv meeting this week. We are postponing until post-stabillity because we will likely want to align with user/identity namespace once that is stable. We think that in most cases it is the |
Area(s)
area:db
Is your change request related to a problem? Please describe.
Today, there is a db.user field under db namespace. However, a recent change proposed to mark the db.user as deprecated and add it back later once nested namespace is supported.
Describe the solution you'd like
Given db.user is currently being used by customers/venders, we would like to use this issue for tracking purpose to make sure db.user won't be removed without a replacement.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: