-
Notifications
You must be signed in to change notification settings - Fork 100
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
net/http
instrumentation does not support Go < 1.19
#1771
Comments
cc @damemi |
The fact this hasn't been reported yet supports us dropping support for Go < 1.19. |
This has been broken for 9 months: #790 That is 8 releases without it being reported. |
+1 I've always been in favor of adopting a reasonable support timeline as an open source project |
+1 on supporting versions >= 1.17. |
SIG meeting notes:
|
Describe the bug
Running the auto-instrumentation for a service using Go < 1.19 will fail to find offsets needed for the
net/http
instrumentation. This causes the auto-instrumentation to crash.Environment
To Reproduce
Update the database end-2-end test:
Run the fixtures test:
make fixture-databasesql
See error:
Expected behavior
Compatibility docs state Go 1.12 is supported. Therefore, this shouldn't fail.
Additional context
Missing offset:
opentelemetry-go-instrumentation/internal/pkg/inject/offset_results.json
Lines 8515 to 8695 in 785339c
Note: there are many other offsets not available for Go 1.12.
The text was updated successfully, but these errors were encountered: