tagarr commented on PR #420:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/420#issuecomment-1310590127

   @morhidi I don't think providing an additional config map to use as an 
override is a solution. OLM already creates a configmap for the config, but 
there is no way to get this dynamic variable into it, other than an init 
container to modify post creation or user modifications post install. The 
easiest way, which is done for pretty much all olm'd operators is by using an 
Env ver injected by these annotations. If you don't want it in the helm chart I 
can remove it, we should be able to add it to the csv when we generate it, 
though it does mean a bit more work. But we do need to env var override


-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to