astefanutti commented on pull request #1861:
URL: https://github.com/apache/camel-k/pull/1861#issuecomment-801931277
Thanks a lot. Here are a couple of points:
- I wonder if we should embed `PodSpec` instead of `PodTemplateSpec`.
Annotations can already be passed via the Integration, and that would avoid
opening the door to `Finalizers`, `OwnerReferences` and the like?
- Should we filter out fields from the embedded `PodTemplateSpec` /
`PodSpec`? Are there any that do not functionally make sense embedding?
- We have to add e2e tests covering the first use cases we have in mind,
like:
- Setting environment variables to the Integration container
- Adding an init or side-car container
- Setting `TopologySpreadConstraints`
- ...
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
