Reduce CI tests flakiness

[Leiyks]

Description

This PR fixes multiple sources of test flakiness in CI by implementing proper retry/wait logic and fixing dependency installation.

1. Laravel Octane Test Flakiness

• Added waitUntilServerRunning() to properly wait for Octane server startup instead of fixed sleep(1)
• Eliminates race condition causing NULL traces when tests run before server is ready

2. SQLSRV Connection Flakiness

• Added retry logic (30 attempts, 500ms delay) for SQL Server connections
• Handles variable container startup time

3. PHP-FPM/Nginx Startup Race Condition

• Added waitUntilServerRunning() to both PhpFpm and NginxServer classes
• Fixed startup order: PHP-FPM starts first (nginx depends on it), then nginx
• Eliminates 502 Bad Gateway errors when tests run before backends are ready

4. System Tests Docker Hub Authentication

• Auto-install unzip (via apt-get/apk) when missing to extract Vault CLI
• Fixes "unzip: not found" error preventing Docker Hub login

5. CentOS Package Verification

• Added retry loop (3 attempts) for yum update
• Handles vault.centos.org reliability issues

[datadog-datadog-prod-us1]

⚠️ Tests

✨ Fix all issues with Cursor

⚠️ Warnings

🧪 1024 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: 6970ca6600000000993eda600631da26
tid: 6970ca6600000000
hexProcessTraceId: 993eda600631da26
hexProcessSpanId: c0af6f7169142aca
processTraceId: 11042503442314680870
processSpanId: 13884438709091445450

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

View all

ℹ️ Info

❄️ No new flaky tests detected

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

[pr-commenter]

Benchmarks [ profiler ]

Benchmark execution time: 2026-01-14 12:51:37

Comparing candidate commit 65e5637 in PR branch leiyks/fix-sqlsrv-and-laravel-octane-flakiness with baseline commit 9c58f4d in branch leiyks/fix-flaky-ci-tests.

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

scenario:php-profiler-timeline-memory-with-profiler-and-timeline

• 🟥 execution_time [+29.593ms; +47.922ms] or [+2.434%; +3.942%]
• 🟩 cpu_usage_percentage [-2.945%; -2.051%]

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.01%. Comparing base (0f0c13a) to head (0f1711b).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3574      +/-   ##
==========================================
+ Coverage   61.91%   62.01%   +0.10%     
==========================================
  Files         140      140              
  Lines       13311    13311              
  Branches     1762     1762              
==========================================
+ Hits         8241     8255      +14     
+ Misses       4280     4267      -13     
+ Partials      790      789       -1     

see 3 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 0f0c13a...0f1711b. 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.

[bwoebi]

Looks good to me now, thanks :-)