Skip to content

Conversation

szegedi
Copy link
Contributor

@szegedi szegedi commented Aug 12, 2025

What does this PR do?

Adds DD_PROFILING_TIMELINE_ENABLED into the list of supported env vars.

Motivation

It was apparently dropped accidentally with the environment variable rework in #5603, so since v5.55.0 DD_PROFILING_TIMELINE_ENABLED=false had no effect and timelines couldn't be disabled.

@szegedi szegedi requested a review from a team as a code owner August 12, 2025 12:07
Copy link

Overall package size

Self size: 11.35 MB
Deduped: 111 MB
No deduping: 111.39 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | @datadog/libdatadog | 0.7.0 | 35.02 MB | 35.02 MB | | @datadog/native-appsec | 10.1.0 | 20.37 MB | 20.37 MB | | @datadog/native-iast-taint-tracking | 4.0.0 | 11.72 MB | 11.73 MB | | @datadog/pprof | 5.9.0 | 9.77 MB | 10.14 MB | | @opentelemetry/core | 1.30.1 | 908.66 kB | 7.16 MB | | protobufjs | 7.5.3 | 2.95 MB | 5.6 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 | | jsonpath-plus | 10.3.0 | 617.18 kB | 1.08 MB | | import-in-the-middle | 1.14.2 | 122.36 kB | 850.93 kB | | lru-cache | 10.4.3 | 804.3 kB | 804.3 kB | | source-map | 0.7.4 | 226 kB | 226 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 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 | 7.0.5 | 63.38 kB | 63.38 kB | | istanbul-lib-coverage | 3.2.2 | 34.37 kB | 34.37 kB | | rfdc | 1.4.1 | 27.15 kB | 27.15 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB | | @isaacs/ttlcache | 1.4.1 | 25.2 kB | 25.2 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | shell-quote | 1.8.3 | 23.74 kB | 23.74 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.4 | 3.96 kB | 3.96 kB |

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

Copy link

codecov bot commented Aug 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.57%. Comparing base (36a2bd1) to head (3b1ff8f).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6264      +/-   ##
==========================================
- Coverage   83.32%   82.57%   -0.76%     
==========================================
  Files         478      471       -7     
  Lines       19824    19667     -157     
==========================================
- Hits        16519    16240     -279     
- Misses       3305     3427     +122     

☔ 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.

Copy link

datadog-official bot commented Aug 12, 2025

⚠️ Tests

⚠️ Warnings

❄️ 2 New flaky tests detected

cypress@latest commonJS intelligent test runner does not skip tests if suite is marked as unskippable from integration-tests/cypress/cypress.spec.js (Datadog)
Cannot read properties of undefined (reading 'content')
cypress@latest commonJS intelligent test runner does not skip tests if test skipping is disabled by the API from integration-tests/cypress/cypress.spec.js (Datadog)
expected 'fail' to equal 'pass'

ℹ️ Info

🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 3b1ff8f | Docs | Was this helpful? Give us feedback!

@szegedi szegedi merged commit 63e70ec into master Aug 12, 2025
688 checks passed
@szegedi szegedi deleted the szegedi/fix-timeline-enablec branch August 12, 2025 12:42
@dd-octo-sts dd-octo-sts bot mentioned this pull request Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants