davsclaus commented on code in PR #24468: URL: https://github.com/apache/camel/pull/24468#discussion_r3530599236
########## dsl/camel-kamelet-main/src/generated/resources/camel-factoryfinder-known-dependencies.properties: ########## @@ -17,6 +17,11 @@ META-INF/services/org/apache/camel/bean-processor-factory=camel:bean META-INF/services/org/apache/camel/bean-proxy-factory=camel:bean +META-INF/services/org/apache/camel/cluster-service-consul=camel:consul Review Comment: Good catch! The factory finder generation in `PrepareKameletMainMojo` was using `findCamelDirectory()` which walks **up** parent directories instead of scanning into nested modules. For `camel-infinispan/camel-infinispan/`, it would find an empty directory at the `components/` level and return no results. Fixed by replacing `findCamelDirectory()` with direct path resolution and adding nested module scanning. This also picked up several other previously-missed factory finders from nested modules (AWS/Azure/Google/IBM secret manager periodic tasks, avro-rpc-jetty, spring-xml, microprofile-config). -- 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]
