[PROF-9960] Use mapping filename instead of comm for function filename (

#21) * Rename comm label into 'thread_name' and add it to custom context * Use mapping filename instead of comm for function filename
DataDog · Jun 26, 2024 · f313755 · f313755
1 parent 4476326
commit f313755
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/reporter/datadog_reporter.go b/reporter/datadog_reporter.go
@@ -316,7 +316,7 @@ func (r *DatadogReporter) reportProfile(ctx context.Context) error {
 
 	tags := strings.Split(config.ValidatedTags(), ";")
 
-	customAttributes := []string{"container_id", "container_name"}
+	customAttributes := []string{"container_id", "container_name", "thread_name"}
 	for _, attr := range customAttributes {
 		tags = append(tags, "ddprof.custom_ctx:"+attr)
 	}
@@ -414,7 +414,7 @@ func (r *DatadogReporter) getPprofProfile() (profile *pprofile.Profile,
 					loc.Mapping = tmpMapping
 				}
 				line := pprofile.Line{Function: createPprofFunctionEntry(funcMap, profile, "",
-					traceInfo.comm)}
+					loc.Mapping.File)}
 				loc.Line = append(loc.Line, line)
 			case libpf.KernelFrame:
 				// Reconstruct frameID
@@ -543,7 +543,7 @@ func createPprofFunctionEntry(funcMap map[funcInfo]*pprofile.Function,
 
 func addTraceLabels(labels map[string][]string, i traceFramesCounts) {
 	if i.comm != "" {
-		labels["comm"] = append(labels["comm"], i.comm)
+		labels["thread_name"] = append(labels["thread_name"], i.comm)
 	}
 
 	if i.podName != "" {