oscerd opened a new pull request, #2962:
URL: https://github.com/apache/camel-kamelets/pull/2962

   Fixes #2954
   
   The Elasticsearch and OpenSearch Kamelets declare the same `enableSSL` 
property at the same position with identical titles and descriptions, but with 
opposite defaults:
   
   | Kamelet | before | after |
   |---|---|---|
   | `elasticsearch-index-sink` | `true` | `true` (unchanged) |
   | `elasticsearch-search-source` | `true` | `true` (unchanged) |
   | `opensearch-index-sink` | `false` | **`true`** |
   | `opensearch-search-source` | `false` | **`true`** |
   
   Given the byte-for-byte similarity of the sibling declarations, the 
divergence looks like drift rather than a deliberate choice.
   
   ### Behaviour change
   
   Existing deployments pointing at a plain-HTTP OpenSearch cluster will need 
`enableSSL=false` set explicitly. `camel-kamelets` has no upgrade guide of its 
own, so the note belongs in the `apache/camel` upgrade guide for the release 
that picks this up — flagging for a maintainer.
   
   Related: #2791.
   
   ### Verification
   
   - `script/validator` reports no errors
   - `mvn verify` passes
   
   ---
   _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]

Reply via email to