gnodet opened a new pull request, #2999: URL: https://github.com/apache/cxf/pull/2999
## Summary - Replace `await().until(() -> spans.size() == N)` with `await().untilAsserted(() -> assertThat(message, size, equalTo(N)))` across all tracing system tests (Brave, OpenTelemetry, OpenTracing; JAX-RS and JAX-WS) - When the Awaitility wait times out, the error message now includes the actual span count **and** the full list of collected spans - This diagnostic improvement will help determine whether flaky failures (see #2971) are caused by: missing spans (count < expected), extra spans from cross-test contamination (count > expected), or a pure timing issue ## Context Related to #2971 and the discussion about root-causing the flaky tracing test failures. @reta pointed out that the thread-safety fix alone may not address all failures, since the OTel tests also fail despite using thread-safe `InMemorySpanExporter`. This PR adds diagnostic output so that when CI fails, we can see exactly what spans were collected and identify the root cause. ## Test plan - [x] Compilation verified locally (`mvn compile test-compile -pl systests/tracing`) - [ ] CI build should pass — no behavioral changes, only improved error messages on failure - [ ] When a tracing test times out in CI, the error message should now show the span list contents _Claude Code on behalf of Guillaume Nodet_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
