oscerd commented on PR #29249: URL: https://github.com/apache/camel/pull/29249#issuecomment-6736201080
@davsclaus @Croway thanks for the reviews. Rebased on main after #29238, with the test-infra metadata regenerated (1ffe1a7e7017); the review points are in 58115fec3209: - **Development notes:** the "Important Development Notes" of `azure-cosmosdb-component.adoc` now say that `mvn verify` runs the integration tests on Floci AZ with Docker, name the 6 tests that run against an account only, and give the `-Dendpoint`/`-DaccessKey` command for Azure. The catalog copy is regenerated. - **TLS:** the client uses the emulator switch of the SDK (`COSMOS.EMULATOR_SERVER_CERTIFICATE_VALIDATION_DISABLED`, plus `COSMOS.EMULATOR_HOST` for a remote Docker host) instead of swapping the JVM trust store, so there is no certificate URL to expose any more. The Javadoc says what a client needs (gateway mode, the switch), and that a route needs its own `cosmosAsyncClient`: [CAMEL-26763](https://issues.apache.org/jira/browse/CAMEL-26763) is about a connection-mode option on the component. - **Remote-only tests:** gated on `azure.instance.type`, the one switch. - **Lease database cleanup:** deletes each database directly and ignores a 404, on a best-effort basis. 16 ITs pass against Floci AZ 0.16.0, 6 are remote only; the 59 unit tests pass. _Claude Code on behalf of Andrea Cosentino_ -- 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]
