FourSpaces opened a new issue, #5212: URL: https://github.com/apache/kyuubi/issues/5212
### Code of Conduct - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct) ### Search before asking - [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues. ### Describe the bug I got this problem when deploying Kyuubi with k8s charts of Kyuubi ### Affects Version(s) 1.7.1 ### Kyuubi Server Log Output ```logtalk 2023-08-29 07:00:05.228 INFO org.apache.kyuubi.zookeeper.EmbeddedZookeeper: Service[EmbeddedZookeeper] is stopped. Exception in thread "main" java.lang.IllegalArgumentException: 'ArrayBuffer(PROMETHEUS, REST, THRIFT_BINARY)' in kyuubi.frontend.protocols is invalid. the frontend protocol should be one or more of THRIFT_BINARY,THRIFT_HTTP,REST,MYSQL,TRINO at org.apache.kyuubi.config.TypedConfigBuilder.$anonfun$checkValue$1(ConfigBuilder.scala:173) at org.apache.kyuubi.config.TypedConfigBuilder.$anonfun$transform$1(ConfigBuilder.scala:167) at scala.Option.map(Option.scala:230) at org.apache.kyuubi.config.ConfigEntryWithDefault.readFrom(ConfigEntry.scala:106) at org.apache.kyuubi.config.KyuubiConf.get(KyuubiConf.scala:99) at org.apache.kyuubi.server.KyuubiServer.frontendServices$lzycompute(KyuubiServer.scala:150) at org.apache.kyuubi.server.KyuubiServer.frontendServices(KyuubiServer.scala:149) at org.apache.kyuubi.service.Serverable.initialize(Serverable.scala:45) at org.apache.kyuubi.server.KyuubiServer.initialize(KyuubiServer.scala:178) at org.apache.kyuubi.server.KyuubiServer$.startServer(KyuubiServer.scala:59) at org.apache.kyuubi.server.KyuubiServer$.main(KyuubiServer.scala:99) at org.apache.kyuubi.server.KyuubiServer.main(KyuubiServer.scala) 2023-08-29 07:00:07.437 INFO org.apache.zookeeper.server.SessionTrackerImpl: SessionTrackerImpl exited loop! 2023-08-29 07:00:07.439 WARN org.apache.kyuubi.zookeeper.EmbeddedZookeeper: Service[EmbeddedZookeeper] is not started(STOPPED) yet. ``` ### Kyuubi Engine Log Output _No response_ ### Kyuubi Server Configurations ```yaml ## Helm chart provided Kyuubi configurations kyuubi.kubernetes.namespace=kyuubi kyuubi.frontend.connection.url.use.hostname=false kyuubi.frontend.thrift.binary.bind.port=10009 kyuubi.frontend.thrift.http.bind.port=10010 kyuubi.frontend.rest.bind.port=10099 kyuubi.frontend.mysql.bind.port=3309 kyuubi.frontend.protocols=PROMETHEUS,REST,THRIFT_BINARY # Kyuubi Metrics kyuubi.metrics.enabled=true kyuubi.metrics.reporters= ## User provided Kyuubi configurations ``` ### Kyuubi Engine Configurations _No response_ ### Additional context It seems that the configuration generated by the k8s charts of Kyuubi is incorrect. I have found the corresponding logic. https://github.com/apache/kyuubi/blob/19d54ec7390a2e74c22aa11bbcc5ac57ff61fa41/charts/kyuubi/templates/_helpers.tpl#L22-L33 The correct logic should exclude 'prometheus'. ``` {{- define "kyuubi.frontend.protocols" -}} {{- $protocols := list }} {{- range $name, $frontend := .Values.server }} {{- if eq $frontend.enabled (ne $name "prometheus") }} {{- $protocols = $name | snakecase | upper | append $protocols }} {{- end }} {{- end }} {{- if not $protocols }} {{ fail "At least one frontend protocol must be enabled!" }} {{- end }} {{- $protocols | join "," }} {{- end }} ``` ### Are you willing to submit PR? - [X] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix. - [ ] No. I cannot submit a PR at this time. -- 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]
