Skip to content

chore(profiling): update libdatadog 26 to 27#3640

Merged
realFlowControl merged 2 commits intomasterfrom
florian/libdatadog-bump-27
Feb 12, 2026
Merged

chore(profiling): update libdatadog 26 to 27#3640
realFlowControl merged 2 commits intomasterfrom
florian/libdatadog-bump-27

Conversation

@realFlowControl
Copy link
Copy Markdown
Member

@realFlowControl realFlowControl commented Feb 11, 2026

Description

This updates libdatadog for the profiler to v.27.0.0

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@realFlowControl realFlowControl force-pushed the florian/libdatadog-bump-27 branch from c12a90e to de84d28 Compare February 11, 2026 14:55
@github-actions github-actions Bot added the profiling Relates to the Continuous Profiler label Feb 11, 2026
@datadog-official
Copy link
Copy Markdown

datadog-official Bot commented Feb 11, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

🧪 1027 Tests failed

    ❄️ Known flaky: telemetry data is received() from com.datadog.appsec.php.integration.TelemetryTests (Fix with Cursor)

    testSearchPhpBinaries from integration.DDTrace\Tests\Integration\PHPInstallerTest (Fix with Cursor)

    testSimplePushAndProcess from laravel-58-test.DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest (Fix with Cursor)

View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8fbeff4 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.20%. Comparing base (b9986f9) to head (8fbeff4).
⚠️ Report is 55 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3640      +/-   ##
==========================================
- Coverage   62.21%   62.20%   -0.02%     
==========================================
  Files         141      141              
  Lines       13387    13387              
  Branches     1753     1753              
==========================================
- Hits         8329     8327       -2     
- Misses       4260     4263       +3     
+ Partials      798      797       -1     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9986f9...8fbeff4. Read the comment docs.

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

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Feb 11, 2026

Benchmarks [ profiler ]

Benchmark execution time: 2026-02-11 17:46:05

Comparing candidate commit 8fbeff4 in PR branch florian/libdatadog-bump-27 with baseline commit b9986f9 in branch master.

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

@realFlowControl realFlowControl marked this pull request as ready for review February 11, 2026 15:37
@realFlowControl realFlowControl requested review from a team as code owners February 11, 2026 15:37
@realFlowControl realFlowControl force-pushed the florian/libdatadog-bump-27 branch from 05e1c97 to 8fbeff4 Compare February 11, 2026 17:26
@realFlowControl realFlowControl merged commit de9c414 into master Feb 12, 2026
2058 of 2068 checks passed
@realFlowControl realFlowControl deleted the florian/libdatadog-bump-27 branch February 12, 2026 06:21
@github-actions github-actions Bot added this to the 1.17.0 milestone Feb 12, 2026
dubloom pushed a commit that referenced this pull request Feb 12, 2026
* chore(profiling): update libdatadog 26 to 27
dubloom added a commit that referenced this pull request Feb 12, 2026
* Adds process tags to profiler uploader

* remove useless utils function

* remove empty lines and fix spelling

* add function to ddtrace.sym

* feat(CI: installer tests): fix installer tests by changing enabling check on appsec extension (#3604)

Signed-off-by: Alexandre Rulleau <[email protected]>

* refactor(profiling): use module globals for ZMM state (#3608)

* refactor(profiling): use module globals for ZMM state

* style: fix clippy warnings

* Apply suggestions from code review

Co-authored-by: Florian Engelhardt <[email protected]>

* docs: note ZTS vs NTS differences

---------

Co-authored-by: Florian Engelhardt <[email protected]>

* refactor(profiling): extract Backtrace type (#3612)

* refactor(profiling): extract Backtrace type

In a future change, this may hold a refcount for another object, so
we need to encapsulate it.

* fix `test_collect_stack_sample` not running

---------

Co-authored-by: Florian Engelhardt <[email protected]>

* Propagate RELIABILITY_ENV_BRANCH to downstream pipeline (#3605)

* Add simple_onboarding_appsec to SSI system tests (#3617)

* Stores remote config requests in request-replayer (#3585)

* feat(profiling): internal metrics for overhead (#3616)

* feat(profiling): internal metrics for overhead

* feat(profiling): move CPU time capture to include serialization for `ddprof_upload` for current profile exported

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* feat(profiling): add CPU time tracking for `ddprof_time` thread

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* feat(profiling): separate CPU time tracking per background thread

Co-authored-by: Claude Opus 4.5 <[email protected]>

---------

Co-authored-by: Florian Engelhardt <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>

* fix(tracing): hook is_internal was backwards (#3625)

* Fix phpt asm standalone tests (#3628)

* fix readme file links (#3610)

* test(language-tests): properly skip online tests and disabled soap_qname_crash.phpt on all version (#3632)

Signed-off-by: Alexandre Rulleau <[email protected]>

* Collect framework endpoints (#3548)

* Only run publishing jobs when all dependent pipelines succeed (#3635)

Signed-off-by: Bob Weinand <[email protected]>

* chore(profiling): update libdatadog to 26 (#3633)

* test(CI): manually handle git operation for windows jobs (#3634)

* test(CI): add aggressive git cleanup on windows runner

Signed-off-by: Alexandre Rulleau <[email protected]>

* test(CI): add manual cleanup in before_script step

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* feat(CI): add healthcheck to SQLSRV server setup (#3619)

* feat(CI): add healthcheck to SQLSRV server setup

Signed-off-by: Alexandre Rulleau <[email protected]>

* chore: add troubleshooting script for SQLSRV

Signed-off-by: Alexandre Rulleau <[email protected]>

* feat: add explicit memory limit and paths

Signed-off-by: Alexandre Rulleau <[email protected]>

* chore: replace sqlsrv docker image

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* fix(CI: test_metrics): add explicit flush in logging (#3637)

* fix(logging): fsync crash logs before _Exit() to prevent data loss

When a SIGSEGV occurs, the signal handler logs "Segmentation fault encountered"
and then calls _Exit() which terminates the process immediately. Without fsync(),
kernel write buffers may not be flushed to disk before termination, causing
a race condition where the error log file is sometimes not created.

This fix adds fsync() on Unix/Linux and _commit() on Windows after write() in
ddtrace_log_with_time() to ensure crash logs persist to disk before process
termination.

The issue affects production (rare but possible during power loss, kernel panic,
or I/O errors) and causes consistent test failures where tests check for log
files immediately after crashes (before kernel writeback completes).

Fixes flaky test_metrics SigSegVTest::testGet failures on Kubernetes where
dd_php_error.log was not being created consistently.

* fix(signals): move flush in sigsegv handler

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* Adds process_tags to live debugger payloads (#3580)

* init process tags for APM

Co-Authored-By: PROFeNoM <[email protected]>

* feat(process_tags): add process_tags to tracing payloads

* small auto review and fix test

* bwoebi review

* fix test

* Adds process_tags to live debugger payloads

* temporary libdatadog bump

* auto review

* bump libdatadog

* fix build

* update makefile && make cbindgen

* fixing test

* fixing test

* fix appsec tests

---------

Co-authored-by: PROFeNoM <[email protected]>

* chore(profiling): update libdatadog 26 to 27 (#3640)

* chore(profiling): update libdatadog 26 to 27

* process tags were removed while rebasing to sign commit

---------

Signed-off-by: Alexandre Rulleau <[email protected]>
Signed-off-by: Bob Weinand <[email protected]>
Co-authored-by: Florian Engelhardt <[email protected]>
Co-authored-by: Alexandre Rulleau <[email protected]>
Co-authored-by: Levi Morrison <[email protected]>
Co-authored-by: Laplie Anderson <[email protected]>
Co-authored-by: Alejandro Estringana Ruiz <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>
Co-authored-by: Bob Weinand <[email protected]>
Co-authored-by: PROFeNoM <[email protected]>
BridgeAR pushed a commit that referenced this pull request Feb 19, 2026
* chore(profiling): update libdatadog 26 to 27
BridgeAR pushed a commit that referenced this pull request Feb 19, 2026
* Adds process tags to profiler uploader

* remove useless utils function

* remove empty lines and fix spelling

* add function to ddtrace.sym

* feat(CI: installer tests): fix installer tests by changing enabling check on appsec extension (#3604)

Signed-off-by: Alexandre Rulleau <[email protected]>

* refactor(profiling): use module globals for ZMM state (#3608)

* refactor(profiling): use module globals for ZMM state

* style: fix clippy warnings

* Apply suggestions from code review

Co-authored-by: Florian Engelhardt <[email protected]>

* docs: note ZTS vs NTS differences

---------

Co-authored-by: Florian Engelhardt <[email protected]>

* refactor(profiling): extract Backtrace type (#3612)

* refactor(profiling): extract Backtrace type

In a future change, this may hold a refcount for another object, so
we need to encapsulate it.

* fix `test_collect_stack_sample` not running

---------

Co-authored-by: Florian Engelhardt <[email protected]>

* Propagate RELIABILITY_ENV_BRANCH to downstream pipeline (#3605)

* Add simple_onboarding_appsec to SSI system tests (#3617)

* Stores remote config requests in request-replayer (#3585)

* feat(profiling): internal metrics for overhead (#3616)

* feat(profiling): internal metrics for overhead

* feat(profiling): move CPU time capture to include serialization for `ddprof_upload` for current profile exported

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* feat(profiling): add CPU time tracking for `ddprof_time` thread

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* feat(profiling): separate CPU time tracking per background thread

Co-authored-by: Claude Opus 4.5 <[email protected]>

---------

Co-authored-by: Florian Engelhardt <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>

* fix(tracing): hook is_internal was backwards (#3625)

* Fix phpt asm standalone tests (#3628)

* fix readme file links (#3610)

* test(language-tests): properly skip online tests and disabled soap_qname_crash.phpt on all version (#3632)

Signed-off-by: Alexandre Rulleau <[email protected]>

* Collect framework endpoints (#3548)

* Only run publishing jobs when all dependent pipelines succeed (#3635)

Signed-off-by: Bob Weinand <[email protected]>

* chore(profiling): update libdatadog to 26 (#3633)

* test(CI): manually handle git operation for windows jobs (#3634)

* test(CI): add aggressive git cleanup on windows runner

Signed-off-by: Alexandre Rulleau <[email protected]>

* test(CI): add manual cleanup in before_script step

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* feat(CI): add healthcheck to SQLSRV server setup (#3619)

* feat(CI): add healthcheck to SQLSRV server setup

Signed-off-by: Alexandre Rulleau <[email protected]>

* chore: add troubleshooting script for SQLSRV

Signed-off-by: Alexandre Rulleau <[email protected]>

* feat: add explicit memory limit and paths

Signed-off-by: Alexandre Rulleau <[email protected]>

* chore: replace sqlsrv docker image

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* fix(CI: test_metrics): add explicit flush in logging (#3637)

* fix(logging): fsync crash logs before _Exit() to prevent data loss

When a SIGSEGV occurs, the signal handler logs "Segmentation fault encountered"
and then calls _Exit() which terminates the process immediately. Without fsync(),
kernel write buffers may not be flushed to disk before termination, causing
a race condition where the error log file is sometimes not created.

This fix adds fsync() on Unix/Linux and _commit() on Windows after write() in
ddtrace_log_with_time() to ensure crash logs persist to disk before process
termination.

The issue affects production (rare but possible during power loss, kernel panic,
or I/O errors) and causes consistent test failures where tests check for log
files immediately after crashes (before kernel writeback completes).

Fixes flaky test_metrics SigSegVTest::testGet failures on Kubernetes where
dd_php_error.log was not being created consistently.

* fix(signals): move flush in sigsegv handler

Signed-off-by: Alexandre Rulleau <[email protected]>

---------

Signed-off-by: Alexandre Rulleau <[email protected]>

* Adds process_tags to live debugger payloads (#3580)

* init process tags for APM

Co-Authored-By: PROFeNoM <[email protected]>

* feat(process_tags): add process_tags to tracing payloads

* small auto review and fix test

* bwoebi review

* fix test

* Adds process_tags to live debugger payloads

* temporary libdatadog bump

* auto review

* bump libdatadog

* fix build

* update makefile && make cbindgen

* fixing test

* fixing test

* fix appsec tests

---------

Co-authored-by: PROFeNoM <[email protected]>

* chore(profiling): update libdatadog 26 to 27 (#3640)

* chore(profiling): update libdatadog 26 to 27

* process tags were removed while rebasing to sign commit

---------

Signed-off-by: Alexandre Rulleau <[email protected]>
Signed-off-by: Bob Weinand <[email protected]>
Co-authored-by: Florian Engelhardt <[email protected]>
Co-authored-by: Alexandre Rulleau <[email protected]>
Co-authored-by: Levi Morrison <[email protected]>
Co-authored-by: Laplie Anderson <[email protected]>
Co-authored-by: Alejandro Estringana Ruiz <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>
Co-authored-by: Bob Weinand <[email protected]>
Co-authored-by: PROFeNoM <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

profiling Relates to the Continuous Profiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants