-
Notifications
You must be signed in to change notification settings - Fork 502
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
Remove instrgen #5530
Labels
area: instrgen
Related to the instrgen package
Comments
1 task
There is an ongoing discussion to donate the golang compile time instrumentation from Alibaba Cloud. We found that the approach shares the similar idea with what InstrGen does and we are working on a solution to combine the two approaches. open-telemetry/community#1961 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It looks like
instrgen
is no longer maintained. #3682 is the last PR which was submitted more than year a ago.I propose removing
instrgen
unless there is someone willing to maintain it.I believe it is safe to remove the module so since it was never released, and we have the code in the Git history if we wish to resurrect the component in the future.
This change will make CI faster and less flaky.
We would also get rid of this PR: #5494.
CC @pdelewski @MrAlias (as codeowners)
The text was updated successfully, but these errors were encountered: