Skip to content

ci(profiling): increase memory limit to prevent OOM during Rust compi…#3647

Draft
Leiyks wants to merge 2 commits intomasterfrom
leiyks/increase-memory-on-profiling-tests
Draft

ci(profiling): increase memory limit to prevent OOM during Rust compi…#3647
Leiyks wants to merge 2 commits intomasterfrom
leiyks/increase-memory-on-profiling-tests

Conversation

@Leiyks
Copy link
Contributor

@Leiyks Leiyks commented Feb 13, 2026

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@github-actions github-actions bot added the profiling Relates to the Continuous Profiler label Feb 13, 2026
…lation

The profiling tests were failing due to OOM kills (signal 9, SIGKILL)
when compiling large Rust crates like syn and regex-syntax with full
optimization (--profile profiler-release --all-features).

The combination of:
- Building on ramdisk (/mnt/ramdisk/cargo)
- Full optimization level
- Parallel compilation of many crates
- Large crates like syn (proc-macro library)

exceeded the 4GB memory limit. Increased to 6GB to provide sufficient
headroom for parallel optimized builds.
@Leiyks Leiyks force-pushed the leiyks/increase-memory-on-profiling-tests branch from b0a4858 to 8a3a4ca Compare February 13, 2026 13:10
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Feb 13, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

🧪 1026 Tests failed

    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)

testSimplePushAndProcess from laravel-8x-test.DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 698f514a000000005d5210d062fd2984
tid: 698f514a00000000
hexProcessTraceId: 5d5210d062fd2984
hexProcessSpanId: 527b9006cc728531
processTraceId: 6724455680817572228
processSpanId: 5943502492119762225
View all

ℹ️ Info

❄️ No new flaky tests detected

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

@codecov-commenter
Copy link

codecov-commenter commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.20%. Comparing base (ee75439) to head (2107d60).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3647      +/-   ##
==========================================
+ Coverage   62.11%   62.20%   +0.08%     
==========================================
  Files         141      141              
  Lines       13387    13387              
  Branches     1753     1753              
==========================================
+ Hits         8315     8327      +12     
+ Misses       4273     4263      -10     
+ Partials      799      797       -2     

see 2 files 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 ee75439...2107d60. 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.

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.

2 participants