Skip to content

fix(ci): skip flaky live-debugger probe tests under valgrind#3947

Draft
Leiyks wants to merge 1 commit into
masterfrom
leiyks/fix-ci-livedebugger-valgrind
Draft

fix(ci): skip flaky live-debugger probe tests under valgrind#3947
Leiyks wants to merge 1 commit into
masterfrom
leiyks/fix-ci-livedebugger-valgrind

Conversation

@Leiyks
Copy link
Copy Markdown
Contributor

@Leiyks Leiyks commented Jun 3, 2026

Skip 5 flaky live-debugger probe tests under valgrind — they exceed the ~15s remote-config probe wait and report LEAK&FAIL. Mirrors the existing skip in debugger_span_probe_class.phpt; tests still run in the normal pass.

…tension_ci]

The valgrind pass in test_extension_ci sets USE_ZEND_ALLOC=0 (via run-tests -m),
which makes the ~15s await_probe_installation window in live_debugger.inc time out,
causing 5 tests to produce empty output and fail with LEAK&FAIL. The same tests pass
in the normal (non-valgrind) pass. Mirrors the existing skip idiom in
debugger_span_probe_class.phpt (line 5).

Affected tests:
- debugger_enable_dynamic_config.phpt
- debugger_log_probe.phpt
- debugger_log_probe_process_tags.phpt
- debugger_span_decoration_probe.phpt
- debugger_span_probe.phpt
@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 Bot commented Jun 3, 2026

Pipelines  Tests

Fix all issues with BitsAI

⚠️ Warnings

🚦 43 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | ASAN test_c with multiple observers: [8.5]   View in Datadog   GitLab

See error Test failure in shm_data_internal_fns.phpt: expected bool(false) but got bool(true) and expected float(-1) but got float(1).

DataDog/apm-reliability/dd-trace-php | benchmarks-tracer   View in Datadog   GitLab

See error Composer could not resolve dependencies for laravel/framework due to security advisories.

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

See error Assertion failed in zend_hash.c:903: '(zend_gc_refcount(&(ht)->gc) == 1) || ((ht)->u.flags & (1<<6))' during installation of debugger metric probe.

View all 43 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🔄 Datadog auto-retried 1 job - 1 passed on retry View in Datadog

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 54.15% (+0.03%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 908ea5c | 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant