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

[I] Camel debug is not working [camel-quarkus]

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

   ### Bug description
   
   - `jbang '-Dcamel.jbang.version=4.6.0' camel@apache/camel init 'Demo.java' `
   - `jbang '-Dcamel.jbang.version=4.6.0' camel@apache/camel export --runtime=quarkus --gav=com.acme:myproject:1.0-SNAPSHOT `
   - `mvn quarkus:dev -Pcamel.debug`
   - Connect with jconsole
   - Go to the Route MBean operations and call `suspend`
   - it i logged `INFO  [org.apa.cam.imp.eng.AbstractCamelContext] (RMI TCP Connection(10)-127.0.0.1) Suspended route1 (timer://java)` but the route is still executed
   
    I tried also with camel.jbang.version 4.7.0-SNAPSHOT which is genrating project with Camel Quarkus 3.12.0


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


Re: [I] Camel debug is not working [camel-quarkus]

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

   Something similar was mentioned elsewhere recently.
   
   It is likely related to CQ configuring [`NoShutdownStrategy`](https://github.com/apache/camel-quarkus/blob/44b5f7adbe88ca571db873fc448c4949f27c6c48/extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelContextProcessor.java#L122) in dev mode. It prohibits route suspension.
   
   You can suppress it by setting a property like `camel.main.shutdownTimeout=30`.
   
   


-- 
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: [I] Camel debug is not working [camel-quarkus]

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

   calling `mvn quarkus:dev -Pcamel.debug -Dcamel.main.shutdownTimeout=30` is effectively working. Thanks for the workaround.
   
   What is the purpose of this NoShutdownStrategy in dev mode?
   And why would it prevent to suspend/stop routes?
   
   


-- 
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: [I] Camel debug is not working [camel-quarkus]

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

   > What is the purpose of this NoShutdownStrategy in dev mode?
   
   I think the original intent was that if you're running in dev mode, you're probably looking to iterate quickly. I.e you don't want hot reloading to a long time due to waiting for inflight messages to complete etc.
   
   > And why would it prevent to suspend/stop routes?
   
   Because it's effectively a [noop implementation](https://github.com/apache/camel-quarkus/blob/87f4d770ed0e8d434efc022b0e86fcbf40a00961/extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/NoShutdownStrategy.java#L27) of `ShutdownStrategy`.
   
   > Would it be possible to modify this strategy when the camel-quarkus-debug depdency is on the classpath?
   
   Yes, that should be possible. Or alternatively, we could implement suspend logic in `NoShutdownStrategy` (maybe by delegating to `DefaultShutdownStrategy`).
   


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