Merge pull request #137 from parca-dev/cleanup

cleanup

Author: gnurizen

Makefile

@@ -141,7 +141,7 @@ support/golbls_1_24.test: ./interpreter/golabels/test/main.go
 support/golbls_cgo.test: ./interpreter/golabels/test/main-cgo.go
 	CGO_ENABLED=1 GOTOOLCHAIN=go1.24.1 go build -ldflags '-extldflags "-static"' -tags $(GO_TAGS),usecgo  -o $@ $<
 
-integration-test-binaries: generate ebpf support/golbls_1_23.test support/golbls_1_24.test support/golbls_cgo.test
+integration-test-binaries: generate ebpf rust-components support/golbls_1_23.test support/golbls_1_24.test support/golbls_cgo.test
 	$(foreach test_name, $(TEST_INTEGRATION_BINARY_DIRS), \
 		(go test -ldflags='-extldflags=-static' -trimpath -c \
 			-tags $(GO_TAGS),static_build,integration \

processmanager/manager.go

@@ -221,19 +221,11 @@ func (pm *ProcessManager) symbolizeFrame(frame int, trace *host.Trace, frames *l
 func (pm *ProcessManager) ConvertTrace(trace *host.Trace) (newTrace *libpf.Trace, err error) {
 	traceLen := len(trace.Frames)
 	kernelFramesLen := len(trace.KernelFrames)
-	log.Debugf("ConvertTrace: traceLen=%d, kernelFramesLen=%d", traceLen, kernelFramesLen)
 
 	newTrace = &libpf.Trace{
 		Frames:       make(libpf.Frames, kernelFramesLen, kernelFramesLen+traceLen),
 		CustomLabels: trace.CustomLabels,
 	}
-
-	// Debug kernel frames
-	for i, kf := range trace.KernelFrames {
-		frame := kf.Value()
-		log.Debugf("ConvertTrace: KernelFrame[%d] type=%d fileID=%x", i, frame.Type, frame.FileID)
-	}
-
 	copy(newTrace.Frames, trace.KernelFrames)
 
 	for i := range traceLen {
@@ -314,14 +306,6 @@ func (pm *ProcessManager) ConvertTrace(trace *host.Trace) (newTrace *libpf.Trace
 			}
 		}
 	}
-
-	// Debug final trace before hashing
-	log.Debugf("ConvertTrace: Final trace has %d frames", len(newTrace.Frames))
-	for i, frameHandle := range newTrace.Frames {
-		frame := frameHandle.Value()
-		log.Debugf("ConvertTrace: Final Frame[%d] type=%d fileID=%x", i, frame.Type, frame.FileID)
-	}
-
 	newTrace.Hash = traceutil.HashTrace(newTrace)
 	return newTrace, nil
 }

tracer/tracer_test.go

@@ -1,6 +1,7 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+// Package tracer contains functionality for populating tracers.
 package tracer // import "go.opentelemetry.io/ebpf-profiler/tracer"
 
 import (