Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check if tracer.delegate is set at tracer.Start to avoid tracking it #569

Merged
merged 2 commits into from
Dec 31, 2023

Conversation

RonFed
Copy link
Contributor

@RonFed RonFed commented Dec 23, 2023

Following #523, this PR adds a check in the tracer.Start function instrumentation to ensure the delegate field is not set. If it does, we abort the rest of the eBPF code and avoid saving irrelevant data.

@RonFed RonFed requested a review from a team December 23, 2023 12:58
@edeNFed edeNFed merged commit da00d2e into open-telemetry:main Dec 31, 2023
16 checks passed
@MrAlias MrAlias mentioned this pull request Jan 2, 2024
@MrAlias MrAlias added this to the v0.10.0-alpha milestone Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants