[KYUUBI #4034] [K8S][HELM] Move helm chart to dedicated directory
### _Why are the changes needed?_ The changes are needed to be compliant with Helm recommendations: 1. https://helm.sh/docs/topics/charts/#the-chart-file-structure ``` The directory name is the name of the chart (without versioning information). Thus, a chart describing WordPress would be stored in a wordpress/ directory. ``` 2. https://helm.sh/docs/howto/chart_releaser_action/#repository-changes ``` The charts should be placed under /charts directory at the top-level of the directory tree. ``` ### _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 Closes #4034 from dnskr/move_helm_chart_to_dedicated_directory. Closes #4034 b6f4b661 [dnskr] [K8S][HELM] Move helm chart to dedicated directory Authored-by: dnskr <dnskrv88@gmail.com> Signed-off-by: ulysses-you <ulyssesyou@apache.org>
This commit is contained in:
parent
70bd55dd68
commit
68bd64c944
@ -35,7 +35,7 @@ create ns kyuubi
|
||||
|
||||
#### Get kyuubi started
|
||||
```bash
|
||||
helm install kyuubi-helm ${KYUUBI_HOME}/docker/helm -n ${namespace_name}
|
||||
helm install kyuubi-helm ${KYUUBI_HOME}/charts/kyuubi -n ${namespace_name}
|
||||
```
|
||||
It will print variables and the way to get kyuubi expose ip and port.
|
||||
```bash
|
||||
|
||||
Loading…
Reference in New Issue
Block a user