search

alibaba / opentelemetry-go-auto-instrumentation

OpenTelemetry Compile-Time Instrumentation for Golang
Apache License 2.0
95 stars 14 forks source link

fix: dont add otel imports manually #39

Closed y1yang0 closed 1 month ago

y1yang0 commented 1 month ago

Please review this patch to remove descrption about adding otel imports manually, that's painful for users.

y1yang0 commented 1 month ago

@123liuziming After several experiments, I am still planning to add otel imports to the main.go file and then perform rule matching, rather than forcing the rule to be matched. This is because the instrument phase relies on RuleBundle, which requires a real matching process, and there is currently no easy way to construct it.