Skip to content

fix(ci): diagnose missing PHP language core dumps - #4081

Draft
realFlowControl wants to merge 2 commits into
masterfrom
florian/no-jira-fix-php-language-core-dumps
Draft

fix(ci): diagnose missing PHP language core dumps#4081
realFlowControl wants to merge 2 commits into
masterfrom
florian/no-jira-fix-php-language-core-dumps

Conversation

@realFlowControl

Copy link
Copy Markdown
Member

Description

Reuse the shared core artifact collector for profiler PHP language tests instead of checking only /usr/local/src/php/core. Record the effective core limit, kernel naming settings, source-directory writability, and disk space before the tests so failures without a dump remain diagnosable. Failed jobs now always have a diagnostic artifact, explicitly report when no ELF core was found, and still generate a GDB report when one is available.

Validation:

  • Generated profiler child-pipeline YAML parses successfully.
  • A forced PHP 8.3 ZTS crash in the CI image produced an ELF core artifact and GDB report.
  • The no-core path produced the diagnostic artifact and explicit message.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@datadog-prod-us1-4

datadog-prod-us1-4 Bot commented Jul 30, 2026

Copy link
Copy Markdown

Pipelines  Tests

Unblock PR with BitsAI

⚠️ Warnings

🚦 7 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [8.1]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [8.2]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [8.4]   View in Datadog   GitLab

View all 7 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: 60.68% (+0.01%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: b36e1b4 | Docs | Datadog PR Page | Give us feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant