csi-driver-lvm icon indicating copy to clipboard operation
csi-driver-lvm copied to clipboard

Bumps versions for k8s-sig images and fixes rbac

Open dmarkwat opened this issue 2 years ago • 1 comments

Support for arm64 was missing due to the use of k8s-sigs images that predated being built cross-platform. The bump in images works on arm64, but only with the additional inclusion of the RBAC change.

This covers create and delete cases for PVCs following the mytest example. Not sure if resize will work; it's possible there's an rbac missing there, too. But if it's anything like the one for the attacher, it's just a missing .../status resource.

Might be worth parameterizing the various k8s-sig images? Happy to do it if anyone thinks that'd be useful.

dmarkwat avatar Oct 04 '21 00:10 dmarkwat

Is the repo behind what's officially published? When adding the chart as dependency, I get the versions in this PR while the repo is way behind.

- name: csi-driver-lvm
  version: "0.4.1"
  repository: "https://helm.metal-stack.io"

If so, can this be merged?

romoh avatar Jun 28 '22 22:06 romoh

Sorry for slow response, your requested changes were already integrated into the chart, which is now located on https://github.com/metal-stack/helm-charts. It works at least with K8s v1.24 now. Thanks for your PR!

Gerrit91 avatar Sep 16 '22 12:09 Gerrit91