From dbb741fc5b0d2fd4b0640ff70ea12aca75864166 Mon Sep 17 00:00:00 2001 From: Cheng Pan Date: Thu, 23 Mar 2023 19:07:57 +0800 Subject: [PATCH] [KYUUBI #4557][HELM] Kyuubi server should bind Pod IP by default ### _Why are the changes needed?_ Fix #4557. ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [x] Add screenshots for manual tests if appropriate After changing, the Kyuubi Server correctly binds the Pod IP. ``` KyuubiTBinaryFrontendService#64 - Starting and exposing JDBC connection at: jdbc:hive2://10.88.224.214:10009/ ``` - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request Closes #4561 from pan3793/helm-host. Closes #4557 018a3a067 [Cheng Pan] [KYUUBI #4557][HELM] Kyuubi server should bind Pod IP by default Authored-by: Cheng Pan Signed-off-by: Cheng Pan --- charts/kyuubi/templates/kyuubi-configmap.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/kyuubi/templates/kyuubi-configmap.yaml b/charts/kyuubi/templates/kyuubi-configmap.yaml index 7a96daaf7..4964e651c 100644 --- a/charts/kyuubi/templates/kyuubi-configmap.yaml +++ b/charts/kyuubi/templates/kyuubi-configmap.yaml @@ -34,7 +34,7 @@ data: kyuubi-defaults.conf: | ## Helm chart provided Kyuubi configurations kyuubi.kubernetes.namespace={{ .Release.Namespace }} - kyuubi.frontend.bind.host=localhost + kyuubi.frontend.connection.url.use.hostname=false kyuubi.frontend.thrift.binary.bind.port={{ .Values.server.thriftBinary.port }} kyuubi.frontend.thrift.http.bind.port={{ .Values.server.thriftHttp.port }} kyuubi.frontend.rest.bind.port={{ .Values.server.rest.port }}