squakez commented on pull request #2771: URL: https://github.com/apache/camel-k/pull/2771#issuecomment-1002132432
I've finished another iteration of development in order to address the points discussed previously. With d85257c we're now validating the input submitted to the trait: we won't allow anything but `configmap` or `secret`. With 78e62f4 we're introducing a new trait, called `mount` which can be used to _mount_ volumes from the different resources we allow. I prefer this name to a generic `resource` as it declare exactly its purpose. The last outstanding point is about garbage collecting the autogenerated configmaps that are used and later removed. Right now they will live beside the Integration, until this one is deleted. I cannot figure out how to manage differently and let them be garbage collected as soon as they are updated (unless we do a manual clean). IMO we can keep them along. @astefanutti please, have a further look and let me know what do you think. -- 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]
