feat(servicegraphs): Add support for db.system.name

CHANGELOG.md

@@ -1319,3 +1319,4 @@ Additionally, default label `span_status` is renamed to `status_code`.
 * [BUGFIX] Increase Prometheus `notfound` metric on tempo-vulture. [#301](https://github.com/grafana/tempo/pull/301)
 * [BUGFIX] Return 404 if searching for a tenant id that does not exist in the backend. [#321](https://github.com/grafana/tempo/pull/321)
 * [BUGFIX] Prune in-memory blocks from missing tenants. [#314](https://github.com/grafana/tempo/pull/314)
+* [ENHANCEMENT] Fix db.system.name attribute not being set in servicegraphs processor. [#4656](https://github.com/grafana/tempo/pull/4656)

modules/generator/processor/servicegraphs/servicegraphs.go

@@ -276,7 +276,7 @@ func (p *Processor) upsertPeerNode(e *store.Edge, spanAttr []*v1_common.KeyValue
 }
 
 // upsertDatabaseRequest handles the logic of adding a database edge on the
-// graph.  If we have a db.name or db.system attribute, we assume this is a
+// graph.  If we have a db.name or db.system or db.system.name attribute, we assume this is a
 // database request.  The name of the edge is determined by the following
 // order:
 //
@@ -300,12 +300,21 @@ func (p *Processor) upsertDatabaseRequest(e *store.Edge, resourceAttr []*v1_comm
 
 	// Check for db.system only if we don't have db.name above
 	if !isDatabase {
-		if _, ok := processor_util.FindAttributeValue(string(semconv.DBSystemKey), resourceAttr, span.Attributes); ok {
+		if name, ok := processor_util.FindAttributeValue(string(semconv.DBSystemKey), resourceAttr, span.Attributes); ok {
+			dbName = name
 			isDatabase = true
 		}
 	}
 
-	// If neither db.system nor db.name are present, we can't determine if this is a database request
+	// Check for db.system.name if neither above is found
+	if !isDatabase {
+		if name, ok := processor_util.FindAttributeValue(string(semconv.DBSystemNameKey), resourceAttr, span.Attributes); ok {
+			dbName = name
+			isDatabase = true
+		}
+	}
+
+	// If neither db.system nor db.name nor db.system.name are present, we can't determine if this is a database request
 	if !isDatabase {
 		return
 	}