Skip to content

ci(profiling): Reduce profiler CI time - #4496

Draft
realFlowControl wants to merge 4 commits into
masterfrom
florian/prof-16656-profiler-ci-build-reuse
Draft

realFlowControl wants to merge 4 commits into
masterfrom
florian/prof-16656-profiler-ci-build-reuse

Conversation

@realFlowControl

@realFlowControl realFlowControl commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Description

PROF-16656 reduces profiler CI time with four changes:

  • Reuse the existing parallel runner for PHP 7.1-7.3 language tests, including PDO redirects. Tests sharing files or ports stay ordered. PHP 7.4+ keeps native parallelism.
  • Increase profiler PHPT jobs from three to five CPUs. Memory and helper resources stay unchanged.
  • Skip unused Rust shared-library links in profiler builds. Keep rlib to avoid extra fat LTO, and preserve the outputs needed by other build callers.
  • Cache Rust dependencies, remove final crate outputs from the cache, and protect cached libraries during PHP's distclean.

In one complete cold-cache comparison, the profiler child pipeline went from 27m on master to 16m, with all 77 jobs passing. The test matrix and reported pass/fail/XFAIL counts were unchanged. Measurements were taken at 81e4a5aa8; CI execution code is unchanged since then.

Five CPUs uses more reserved core-minutes in the CPU-only comparison. Cache transfer can make cold jobs slower.

The PHP 8.6 module-API mismatch also occurs in master's randomized API test and is not addressed here.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@datadog-prod-us1-5

datadog-prod-us1-5 Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Pipelines  Tests

✨ Unblock PR with BitsAI

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 26 Pipeline jobs failed

DDCI - DataDog/dd-trace-php | ASAN Internal api randomized tests: [8.6, amd64] — 🔧 Needs a code fix, caused by this PR

View more details · View in DDCI

DDCI - DataDog/dd-trace-php | ASAN Opcache tests: [8.6] — 🔧 Needs a code fix, caused by this PR

View more details · View in DDCI

DDCI - DataDog/dd-trace-php | Feature flags tests: [8.6] — 🔧 Needs a code fix, caused by this PR

View more details · View in DDCI

View all 26 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 80.13% (+0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 712a444 | Docs | View more details | Give us feedback!

@realFlowControl realFlowControl changed the title Reduce profiler CI time ci(profiling): Reduce profiler CI time Nov 13, 2026
@realFlowControl
realFlowControl force-pushed the florian/prof-16656-profiler-ci-build-reuse branch from 81e4a5a to 712a444 Compare November 13, 2026 19:40
@github-actions github-actions Bot added the profiling Relates to the Continuous Profiler label Nov 13, 2026
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.

1 participant