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
Replace Go version 1.18+1.19 with 1.19+1.22.0 #2572
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 Thanks for the fix 🙏 . Could you use 1.22
instead of 1.22.0
following the pattern from before? If possible of course. By default it should make it use the last patch version of 1.22 instead of strictly using 1.22.0 which is beneficial for us .
The issue you are seeing with appsec was fixed in 2.3.1 of go-libddwaf and it was updated upstream (upstream being dd-trace-go) last week so updating your branch should fix your issue |
Hi Eliott, It is nice to meet you!
I checked https://go.dev/dl/, and I think we actually need to update 1.19 to 1.19.13, as it seems that till 1.21.0 Google published releases like 1.19.0 as 1.19 (so 1.19 means 1.19.0 rather than 1.19.*; quite confusing). Will do it and also check if AppSec now works correctly. |
BenchmarksBenchmark execution time: 2024-02-26 14:39:34 Comparing candidate commit 0557088 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 40 metrics, 1 unstable metrics. |
All good after rebasing, no more AppSec errors! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 💯
What does this PR do?
Updates Go version for macrobenchmark runs from 1.18 + 1.19 to 1.19 + 1.22.0.
Tracer and profiler both work correctly after the change.
AppSec doesn't work in 1.22.0
For macrobenchmark 1.19 all 3 products work correctly.
Motivation
Since 9th of January we have a broken macrobenchmark for Go tracer 1.18. The analysis of error has shown that some dependencies now require Go version 1.19 to compile the latest dd-trace-agent.
We need to fix macrobenchmark runs, so they continue to report relevant information.
Reviewer's Checklist
For Datadog employees:
@DataDog/security-design-and-guidance
.Unsure? Have a question? Request a review!