-
Notifications
You must be signed in to change notification settings - Fork 320
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
v4.1.1 #3221
v4.1.1 #3221
Conversation
* Add Datadog APM Test Agent to handle all plugin test traces
Co-authored-by: Dominik Krejcik <[email protected]>
…bles (#3055) * Include send-data missing headers * Rename DD_TELEMETRY_DEBUG_ENABLED as DD_TELEMETRY_DIAGNOSTIC_LOG_COLLECTION_ENABLED * Move DD_TELEMETRY_HEARTBEAT_INTERVAL to config.js * Remove DD_TELEMETRY_DIAGNOSTIC_LOG_COLLECTION_ENABLED
* Add ability to create and publish .deb and .rpm packages * only look at version tags * add verbose flag
- previously, when encountering a dead code path that requires a not-installed instrumented package, build would fail - this would happen when, say, `knex` requires the `tedious` library for an app that is only making use of `pg` - without `dd-trace/esbuild`, a user simply adds `tedious` to their `external` list and goes on with their day - or in other words, vanilla esbuild doesn't really care when it encounters these missing modules - with `dd-trace/esbuild`, we would throw an error and the build fails - one solution would be to not instrument external packages but many users expect this behavior to work - in fact, we've been telling users to do just this before we supported a plugin - now, with this change, the `require('unused-module')` call remains in the output code - print a warning when this happens (at build time), regardless of debug level, since it might not be intentional
* Fix OracleDB plugin job on CI to use Github Actions services container for Test-Agent instead of docker-compose
BenchmarksComparing candidate commit 30ff2f9 in PR branch Found 0 performance improvements and 8 performance regressions! Performance is the same for 434 metrics, 30 unstable metrics. scenario:startup-control-everything-16
scenario:startup-with-tracer-everything-16
scenario:startup-control-everything-18
scenario:startup-with-tracer-everything-18
|
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.
LGTM for 4.x, though the corresponding 2.x and 3.x proposal PRs seem to have some failures. Should probably sort those before landing this. 🤔
Overall package sizeSelf size: 4.25 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report
@@ Coverage Diff @@
## v4.x #3221 +/- ##
=======================================
Coverage ? 85.76%
=======================================
Files ? 182
Lines ? 7229
Branches ? 33
=======================================
Hits ? 6200
Misses ? 1029
Partials ? 0 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Bug Fixes
Improvements