Skip to content

v5.51.1 proposal #5664

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

Closed
wants to merge 6 commits into from
Closed

v5.51.1 proposal #5664

wants to merge 6 commits into from

Conversation

dd-trace-js[bot]
Copy link

@dd-trace-js dd-trace-js bot commented May 5, 2025

  • [f9b709b11b] - (SEMVER-PATCH) Revert "node action default version" (Roch Devost) #5675
  • [9721d0895f] - (SEMVER-PATCH) switch to default github token for ci visibility benchmark (Roch Devost) #5639
  • [b1727dd9bf] - (SEMVER-PATCH) [Code Origin for Spans] Allow code origin to be disabled (Thomas Watson) #5635
  • [54c86664a9] - (SEMVER-PATCH) use a fixed version for latest in node action (Roch Devost) #5671
  • [108c0b4fba] - (SEMVER-PATCH) fix release proposal not closing patch pr on minor (Roch Devost) #5663

Copy link

github-actions bot commented May 5, 2025

Overall package size

Self size: 9.3 MB
Deduped: 102.55 MB
No deduping: 103.07 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | @datadog/libdatadog | 0.5.1 | 29.73 MB | 29.73 MB | | @datadog/native-appsec | 8.5.2 | 19.33 MB | 19.34 MB | | @datadog/native-iast-taint-tracking | 3.3.1 | 13.99 MB | 13.99 MB | | @datadog/pprof | 5.7.1 | 9.51 MB | 9.88 MB | | @opentelemetry/core | 1.30.1 | 908.66 kB | 7.16 MB | | protobufjs | 7.4.0 | 2.77 MB | 5.42 MB | | @datadog/wasm-js-rewriter | 4.0.1 | 2.85 MB | 3.58 MB | | @datadog/native-metrics | 3.1.1 | 1.02 MB | 1.43 MB | | @opentelemetry/api | 1.8.0 | 1.21 MB | 1.21 MB | | import-in-the-middle | 1.13.1 | 117.64 kB | 839.26 kB | | source-map | 0.7.4 | 226 kB | 226 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | lru-cache | 7.18.3 | 133.92 kB | 133.92 kB | | pprof-format | 2.1.0 | 111.69 kB | 111.69 kB | | @datadog/sketches-js | 2.1.1 | 109.9 kB | 109.9 kB | | lodash.sortby | 4.7.0 | 75.76 kB | 75.76 kB | | ignore | 5.3.2 | 53.63 kB | 53.63 kB | | istanbul-lib-coverage | 3.2.0 | 29.34 kB | 29.34 kB | | rfdc | 1.4.1 | 27.15 kB | 27.15 kB | | @isaacs/ttlcache | 1.4.1 | 25.2 kB | 25.2 kB | | dc-polyfill | 0.1.8 | 25.08 kB | 25.08 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | shell-quote | 1.8.2 | 23.54 kB | 23.54 kB | | limiter | 1.1.5 | 23.17 kB | 23.17 kB | | retry | 0.13.1 | 18.85 kB | 18.85 kB | | semifies | 1.0.0 | 15.84 kB | 15.84 kB | | jest-docblock | 29.7.0 | 8.99 kB | 12.76 kB | | crypto-randomuuid | 1.0.0 | 11.18 kB | 11.18 kB | | ttl-set | 1.0.0 | 4.61 kB | 9.69 kB | | mutexify | 1.4.0 | 5.71 kB | 8.74 kB | | path-to-regexp | 0.1.12 | 6.6 kB | 6.6 kB | | koalas | 1.0.2 | 6.47 kB | 6.47 kB | | module-details-from-path | 1.0.3 | 4.47 kB | 4.47 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

Copy link

codecov bot commented May 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (v5.x@d72d605). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             v5.x    #5664   +/-   ##
=======================================
  Coverage        ?   79.04%           
=======================================
  Files           ?      513           
  Lines           ?    23407           
  Branches        ?        0           
=======================================
  Hits            ?    18501           
  Misses          ?     4906           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented May 5, 2025

Datadog Report

Branch report: v5.51.1-proposal
Commit report: cb44541
Test service: dd-trace-js-integration-tests

✅ 0 Failed, 935 Passed, 0 Skipped, 17m 8.18s Total Time

@pr-commenter
Copy link

pr-commenter bot commented May 5, 2025

Benchmarks

Benchmark execution time: 2025-05-07 05:18:02

Comparing candidate commit 4e9da0c in PR branch v5.51.1-proposal with baseline commit d72d605 in branch v5.x.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1270 metrics, 53 unstable metrics.

rochdev and others added 5 commits May 7, 2025 05:06
Due to a bug in how the tracer configuration was parsed, older versions of the
tracer would always have Code Origin for entry spans enabled, no matter if it
was disabled in config (which it was supposed to be by default). This bug only
applied to Code Origin for entry spans on Fastify, whereas Code Origin for exit
spans correctly adhered to the config.

This bug has been precent in the tracer since Code Origin for Spans was added
in v5.24.0 on October 17th 2024 and there has not been any reports of issues
related to this. Based on that, Code Origin for Spans will now be enabled by
default. The main functional difference with this commit, is that the feature
can now actually be turned off using the config.

Code Origin for exit spans are still experimental:

Since Code Origin for exit spans was never affected by this bug, we have
decided to leave this feature as opt-in, which means it's going to be off by
default. A new experimental config option has been introduced to allow users to
enable/disable Code Origin for exit spans independently, either via an
environment variable:

    DD_CODE_ORIGIN_FOR_SPANS_EXPERIMENTAL_EXIT_SPANS_ENABLED=true

Or via inline config:

    tracer.init({
      codeOriginForSpans: {
        experimental: {
          exit_spans: { enabled: true }
        }
      }
    })
@dd-trace-js dd-trace-js bot force-pushed the v5.51.1-proposal branch from ebd46e3 to 4e9da0c Compare May 7, 2025 05:06
Copy link
Author

dd-trace-js bot commented May 8, 2025

Superseded by #5684.

@dd-trace-js dd-trace-js bot closed this May 8, 2025
@dd-trace-js dd-trace-js bot deleted the v5.51.1-proposal branch May 8, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants