gnodet opened a new pull request, #24617:
URL: https://github.com/apache/camel/pull/24617
## Summary
_Claude Code on behalf of gnodet_
Fix flaky `DynamicRouteIT` (3.8% failure rate — 10 failed, 8 flaky out of
476 CI runs in the last 2 weeks).
**Root cause**: Race between route `Started` status and JMS consumer
readiness. The test sends request-reply messages to `jms:queue:request`
immediately without waiting for the JMS consumer route to be fully subscribed
to the broker. If the consumer is not ready when the first message arrives, the
20s request-reply timeout expires and the test fails.
**Fix**:
- Add `routeId("jmsConsumer")` to the JMS consumer route for referencing
- Use `JmsTestHelper.waitForJmsConsumerRoutes(context, "jmsConsumer")`
before sending messages — the same pattern used in CAMEL-21438 and CAMEL-23980
- Rename generic queue names (`request`, `bar`) to test-specific names
(`DynamicRouteIT.request`, `DynamicRouteIT.reply`) to avoid cross-test message
interference on the shared embedded Artemis broker
## Test plan
- [x] `mvn verify -pl components/camel-jms -Dit.test=DynamicRouteIT` — 2/2
tests pass
- [ ] CI green on this PR
Co-authored-by: Claude Opus 4.6 <[email protected]>
--
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]
