astefanutti opened a new pull request #2881:
URL: https://github.com/apache/camel-k/pull/2881


   This PR fixes an issue introduced with #2814, that removes nil objects from 
the apply patches, that are computed to install the Kamelets from the embedded 
catalog. This breaks Kamelets that use YAML DSL with empty objects, e.g.:  
   
   ```yaml
     flow:
       from:
         steps:
         - marshal:
             json: {}
   ```
   
   or:
   
   ```yaml
     flow:
       from:
         steps:
           - filter:
             steps:
               - stop: {}
   ```
   
   Positive patch is only needed when applying structured objects, whose 
initialised fields would take ownership even for non-managed fields. The 
"plain" patch can be used when applying unstructured objects. So the PR moves 
to installing the Kamelets as unstructured objects, so as to maintain the exact 
definition from the `.spec.flow` field, that's encoded as a `RawMessage` 
struct. 
   
   **Release Note**
   ```release-note
   NONE
   ```
   


-- 
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: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to