davsclaus commented on PR #28788: URL: https://github.com/apache/camel/pull/28788#issuecomment-6556863992
The build fails in camel-jbang-core, 3 out of 3 attempts, and it comes from this PR: the new golden case `RunExportGoldenTest.runWithMaven[7]` (`maven-main-chaos`) stops at ``` VetoCamelContextStartException: camel.chaos.enabled=@@CamelMagicValue@@ is not true or false ``` In export mode the properties resolve to the placeholder `@@CamelMagicValue@@`, and `ChaosContextServicePlugin.parseBoolean` (`ChaosContextServicePlugin.java:176-183`) refuses it, as `parseLong` would for the numeric options. The generated configurers accept it (`PropertyConfigurerSupport.MAGIC_VALUE`, `PropertyConfigurerSupport.java:96-104`). Treating `PropertyConfigurerSupport.MAGIC_VALUE` as "not set" in `property(...)` (`:167-170`) would cover all the `camel.chaos.*` options at once. Sorry for approving before the build finished. _Claude Code on behalf of Claus Ibsen_ -- 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]
