search

metal-stack / helm-charts

Contains metal-stack Helm charts
MIT License
2 stars 10 forks source link

Add support for Talos OS #64

Closed gerhard closed 1 year ago

gerhard commented 1 year ago

Resolves https://github.com/metal-stack/csi-driver-lvm/issues/86

/etc on Talos OS is read-only, which means that this chart will fail to install as is. The new hostWritePath option allows the default /etc/lvm to be overwritten. For Talos OS, setting hostWritePath: /var/etc/lvm fixes the install issue. This is just half the story.

After this installs, creating a PVs will fail since the controller has some hard-coded paths starting with /etc. Here is the second half of this fix: https://github.com/metal-stack/csi-driver-lvm/pull/87

First step is to get that PR 👆 merged first, then update the image versions in this one before merging this change.

Learn more about the Talos' file system: https://www.talos.dev/v1.4/learn-more/architecture/

gerhard commented 1 year ago

I finally had a spare cycle to come back to this. On the positive side, I was able to test this change in production for a few months now. All working great!

Looking forward to getting this merged & a new helm chart version released so that I can stop using my fork of the chart.

majst01 commented 1 year ago

some linter errors occured ?

gerhard commented 1 year ago

some linter errors occured ?

Fixed!