Elaborate more on Local Path Provider #189
Conversation
This mentiones the use of --default-local-storage-path and how to override the default config that comes with k3s. Signed-off-by: James Tocknell <[email protected]>
Add note that /var/lib/rancher/k3s/storage is the default storage location. Signed-off-by: James Tocknell <[email protected]>
|
|
||
| Below we provide a simple example using a PV and PVC. For more details on the usage of the Local Path Provisioner please reference the official documentation [here](https://github.com/rancher/local-path-provisioner/blob/master/README.md#usage). | ||
|
|
||
| K3s allows for overriding the path used on nodes via `--default-local-storage-path` (see the [Server CLI options](../cli/server.md) for more details including defaults), and further changes to Local Path Provisioner can be made via overriding the provided ConfigMap using [HelmChartConfig](../helm/helm#customizing-packaged-components-with-helmchartconfig). The contents of this ConfigMap is detailed [here](https://github.com/rancher/local-path-provisioner/blob/master/README.md#configuration) |
There was a problem hiding this comment.
unfortunately this isn't true. it's not packaged as a helm chart, so you can't configure it as such. The only way to modify it is to start k3s with --disable=local-storage and provide your own deployment.
There was a problem hiding this comment.
Oh, OK. I'll look at rewording this to explain that you need to use your own deployment. Is there a reason that local-path-provisioner wasn't set up via helm, and is there anything stopping the switch being made?
There was a problem hiding this comment.
Historical reasons mostly. Things that we bundled before adding the embedded helm controller were all managed as flat manifests. When time came to add traefik, it was only available as a helm chart, so we added the helm controller. We added HelmChartConfig support some time later. It's somewhat non-trivial to have helm "take over" existing resources in a nondisruprive way, so we've not prioritized migrating things.
Co-authored-by: Brad Davidson <[email protected]> Signed-off-by: James Tocknell <[email protected]>
Co-authored-by: Brad Davidson <[email protected]> Signed-off-by: James Tocknell <[email protected]>
|
You can rebase of master to fix the codespelling CI. Or just ignore it. The test deployment failures must be addresses however. |
Closes #184.