christophd commented on PR #4808: URL: https://github.com/apache/camel-k/pull/4808#issuecomment-1752631016
@squakez yeah I should have looked into PR #4797 first. makes more sense now, thx I had a look into the sanity checks on properties and names that is done on Kamelet reconciliation and it is really quite basic at the moment. I think it only makes sure to not use `source` or `sink` as a Kamelet name and the Kamelet properties should not use `id` as a name. I think we should add this to the documentation instead Also I have had a look into the properties computation done as part of the reconciliation loop and to be honest I have no idea where this should be used in the Kamelet status. Maybe we do a sync with tooling (e.g. Kaoto, ODC) if this is being used but in case the configuration property mechanism is going to change as done in PR #4797 I do not see any use case for that property list in Kamelet status anymore. So finally I am ok with removing the Kamelet reconciliation -- 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]
