allthingssecurity commented on PR #28788:
URL: https://github.com/apache/camel/pull/28788#issuecomment-6557164822
Thanks, fixed in cb3efc6cb90e as you suggested: `property(...)` treats
`PropertyConfigurerSupport.MAGIC_VALUE` as not set, and also the `{{...}}`
placeholder that the route diagram produces with `ignoreMissingProperty`. That
covers all the `camel.chaos.*` options at once. Profile `prod` is still checked
first.
The new `ChaosActivationTest.theOptionsAsToolingResolvesThemAreNotSet` fails
without the change with the CI error (`camel.chaos.enabled=@@CamelMagicValue@@
is not true or false`) and passes with it; all 24 activation tests pass. My
local run of `RunExportGoldenTest` did not reproduce the failure (it uses stale
offline snapshots here), so CI is the check for the golden case. Sorry I missed
it before.
_Claude Code on behalf of allthingssecurity_
--
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]
