TheNetherWatcher commented on PR #6913: URL: https://github.com/apache/camel-k/pull/6913#issuecomment-6828886536
Thanks @squakez, understood. While #6942 and the v3 question are open, would it help to narrow this PR to the part that doesn't depend on bean discovery? That would mean removing the host lookup from the connection factory bean properties and keeping: - `queueName` mapped from the `queues` endpoint parameter (single queue only) - `host` taken from an explicit `host` endpoint parameter when the user sets one, otherwise left to `keda.autoMetadata` as for the other scalers - the e2e test, which already provides `host`, `mode` and `value` through `autoMetadata` Host discovery could then be added on top of the bean metadata from #6942 once that direction is settled. Happy to keep it on hold instead if you prefer. _Claude Code on behalf of @TheNetherWatcher_ -- 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]
