Croway commented on PR #20767: URL: https://github.com/apache/camel/pull/20767#issuecomment-3743030707
@pkalsi97 no worries! moreover, I like the idea yeah, mocking the JSON sounds good, this way we won't have to to download all the images on the CI. We have to make sure that all the properties in the test-infra services like https://github.com/apache/camel/blob/main/test-infra/camel-test-infra-kafka/src/main/java/org/apache/camel/test/infra/kafka/services/KafkaInfraService.java are either supported by the component as is (brokers in this case), or somehow handled, in this case `getBootstrapServers` is not a supported property by the component. I deprecated some of those in the past, but as you can see from the Kafka, I didn't deprecate all of them :), brokers and getBootstrapServers contain the same value at the moment. -- 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]
