You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "gnodet (via GitHub)" <gi...@apache.org> on 2026/07/12 07:09:46 UTC

[PR] CAMEL-24017: Fix flaky DynamicRouteIT [camel]

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 <no...@anthropic.com>


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-24017: Fix flaky DynamicRouteIT [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #24617:
URL: https://github.com/apache/camel/pull/24617#issuecomment-4950331310

   :star2: Thank you for your contribution to the Apache Camel project! :star2:
     :robot: CI automation will test this PR automatically.
   
     :camel: Apache Camel Committers, please review the following items:
   
     * First-time contributors **require MANUAL approval** for the GitHub Actions to run
     * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot although they are normally detected and executed by CI.
     * You can label PRs using `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR.
     * Build and test logs are available in the summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary.
     
     :warning: Be careful when sharing logs. Review their contents before sharing them publicly.


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-24017: Fix flaky DynamicRouteIT [camel]

Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet merged PR #24617:
URL: https://github.com/apache/camel/pull/24617


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org