You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/03/23 11:08:08 UTC
[kyuubi] branch master updated: [KYUUBI #4557][HELM] Kyuubi server should bind Pod IP by default
This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/master by this push:
new dbb741fc5 [KYUUBI #4557][HELM] Kyuubi server should bind Pod IP by default
dbb741fc5 is described below
commit dbb741fc5b0d2fd4b0640ff70ea12aca75864166
Author: Cheng Pan <ch...@apache.org>
AuthorDate: Thu Mar 23 19:07:57 2023 +0800
[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 <ch...@apache.org>
Signed-off-by: Cheng Pan <ch...@apache.org>
---
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 }}