haowang-pony commented on issue #710:
URL:
https://github.com/apache/apisix-ingress-controller/issues/710#issuecomment-942904222
> > > Using command is OK, but it will make the deployment yaml a bit more
complicated. What we use in the helm chart is configmap for configuration file,
I don't know why you didn't use it?
> >
> >
> > As for helm chart, I used it when I did poc of apisix. However, as for
production use, our company prefer to use deployment yaml directly.
> > As for configMap, actaully my first choice is configMap. However, just
as I said in the begainning, when I mount the configMap as config.yaml, the
config.yaml couldn't set the `${ADMIN_PASSWORD}` from environment variable. And
for production use, considering security, we definitely don't set the
`default_cluster_admin_key` directly in configMap. Therefore I have to use
command. It allow me to use `${ADMIN_PASSWORD}` from environment variable
>
> OK, then I think it can be specified temporarily by command. Another way
is to build a docker image (`FROM apisix-ingress-controller`), add a startup
script, and use the `envsubst` tool in the script to replace the environment
variables in the configuration file.
This is a very good suggestion! Thanks a lot!
--
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]
