You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "JiriOndrusek (via GitHub)" <gi...@apache.org> on 2023/04/03 14:12:19 UTC

[GitHub] [camel-quarkus] JiriOndrusek opened a new issue, #4736: [camel-main] Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails with current Camel main

JiriOndrusek opened a new issue, #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736

   Tast [RouteConfigurationsTest#sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration](https://github.com/apache/camel-quarkus/blob/main/integration-test-groups/foundation/route-configurations/src/test/java/org/apache/camel/quarkus/core/it/routeconfigurations/RouteConfigurationsTest.java#L93) fails.
   
   The issue is caused by the fact, that the `onException` condition defined in [xml](https://github.com/apache/camel-quarkus/blob/main/integration-test-groups/foundation/route-configurations/src/main/resources/xml/routeConfigurations.xml#L21), is not triggered. Interesting fact is that if the xml condition is used from different route, or if the route uses different routeCondition (define by java or yaml), the test is succesful.


-- 
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.apache.org

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


[GitHub] [camel-quarkus] JiriOndrusek commented on issue #4736: Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails

Posted by "JiriOndrusek (via GitHub)" <gi...@apache.org>.
JiriOndrusek commented on issue #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736#issuecomment-1723386432

   Fixed in camel-main branch


-- 
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


[GitHub] [camel-quarkus] aldettinger commented on issue #4736: [camel-main] Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails with current Camel main

Posted by "aldettinger (via GitHub)" <gi...@apache.org>.
aldettinger commented on issue #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736#issuecomment-1497761911

   They share the same context, so likely the route configurations are shared also.
   Maybe this is a behavior that is changing with camel 4 ?
   
   @davsclaus @oscerd  What do you think ?


-- 
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


[GitHub] [camel-quarkus] JiriOndrusek closed issue #4736: Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails

Posted by "JiriOndrusek (via GitHub)" <gi...@apache.org>.
JiriOndrusek closed issue #4736: Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails
URL: https://github.com/apache/camel-quarkus/issues/4736


-- 
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


[GitHub] [camel-quarkus] aldettinger commented on issue #4736: [camel-main] Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails with current Camel main

Posted by "aldettinger (via GitHub)" <gi...@apache.org>.
aldettinger commented on issue #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736#issuecomment-1499322985

   Many thanks, so we'll need to fix this when we have bandwidth. And maybe even have the test in camel itself.


-- 
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


[GitHub] [camel-quarkus] davsclaus commented on issue #4736: [camel-main] Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails with current Camel main

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on issue #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736#issuecomment-1499288070

   yes route configurations are global in the same single context. What is triggered is dependent on route configuration ids, and what has been configured on the route.


-- 
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


[GitHub] [camel-quarkus] jamesnetherton commented on issue #4736: [camel-main] Foundation: RouteConfigurationsTest.sendContentToXmlRouteShouldTriggerOnExceptionInYamlRouteConfiguration fails with current Camel main

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #4736:
URL: https://github.com/apache/camel-quarkus/issues/4736#issuecomment-1497515602

   I think this is the change that causes the problem:
   
   https://github.com/apache/camel/commit/15a2ef2948c6bcb38c7310083a565db351c481d1
   
   I don't know if this test scenario is valid or not. Where you define a route configuration in one DSL language and reference it in another (E.g XML -> YAML). Maybe @aldettinger has an opinion.


-- 
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