pan3793 commented on PR #5281: URL: https://github.com/apache/kyuubi/pull/5281#issuecomment-1769827545
> Also, what do you think about excluding the chart from Kyuubi release and creating dedicated release process for the chart? @dnskr the idea looks cool, but I have some concerns about hadoop/hive configuration injecting for using the published helm chart to deploy. let me share the internal practice from my side: even though we deploy Kyuubi/Spark on K8s, the Kyuubi server and Spark application still require to communicate with external HMS, HDFS, HBase, etc., thus bunches of `*-site.xml`(usually has hundreds even thousands lines per file) are required, it's not elegant to inline those XML into `values.yaml`, eventually， we choose to use [Accessing Files Inside Templates](https://helm.sh/docs/chart_template_guide/accessing_files/) as our solution, but as the docs said, it has some limitation, maybe for security purpose, which let us must deploy the helm application from helm chart source, instead of published helm chart from repo. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
