dnskr opened a new pull request, #4147:
URL: https://github.com/apache/kyuubi/pull/4147
### _Why are the changes needed?_
The changes allow to configure multiple frontends in the helm chart.
Notes:
- Removed unused code
- `server.confDir` renamed to `kyuubiConfDir`
- `server.conf` renamed to `kyuubiConf`
- `livenessProbe` and `readinessProbe` changed to execute `bin/kyuubi
status` which actually prints `Kyuubi is not running`. The issue needs to be
reviewed. Also `livenessProbe` and `readinessProbe` should be revisited in the
next PR.
### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including
negative and positive cases if possible
- [ ] Add screenshots for manual tests if appropriate
- [ ] [Run
test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests)
locally before make a pull request
--
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: notifications-unsubscr...@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org
For additional commands, e-mail: notifications-h...@kyuubi.apache.org
