Remove redis database index from redis span name (#1565)

Co-authored-by: Liudmila Molkova <limolkova@microsoft.com>
This commit is contained in:
Trask Stalnaker 2024-11-13 06:30:17 -08:00 committed by GitHub
parent 62306fc875
commit 56a2bb7d0e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 28 additions and 0 deletions

22
.chloggen/1565.yaml Normal file
View File

@ -0,0 +1,22 @@
# Use this changelog template to create an entry for release notes.
#
# If your change doesn't affect end users you should instead start
# your pull request title with [chore] or use the "Skip Changelog" label.
# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: breaking
# The name of the area of concern in the attributes-registry, (e.g. http, cloud, db)
component: db
# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Remove redis database index from the redis span name.
# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
# The values here must be integers.
issues: [ 1449 ]
# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

View File

@ -10,6 +10,12 @@ The Semantic Conventions for [Redis](https://redis.com/) extend and override the
`db.system` MUST be set to `"redis"` and SHOULD be provided **at span creation time**. `db.system` MUST be set to `"redis"` and SHOULD be provided **at span creation time**.
## Span name
Redis spans SHOULD follow the general [database span name convention](./database-spans.md#name),
except that `db.namespace` SHOULD NOT be used in the span name since it is a numeric value that ends up
looking confusing.
## Attributes ## Attributes
<!-- semconv span.db.redis.client --> <!-- semconv span.db.redis.client -->