christian-schlichtherle / synology-csi-chart

A Helm chart for installing the Synology CSI Driver in a Kubernetes cluster.
Apache License 2.0
51 stars 21 forks source link

helm upgrade removing client-info secret #10

Closed beyondwatts closed 1 year ago

beyondwatts commented 1 year ago

I am trying to change the synology/synology-csi to be a local docker build of the latest code as there is a fix Mount issue in multinodes cluster that I'd like to apply...

I've changed the chart values to point to a local image but when I run helm update the client-info secret is being removed...

helm diff upgrade synology-csi -n synology-csi --values synology-csi-chart-values.yml ./container-storage-interface/synology-csi-chart

shows

synology-csi, synology-csi-client-info, Secret (v1) has been removed:
- # Source: synology-csi/templates/client-info.yaml
- apiVersion: v1
- kind: Secret
- metadata:
-   labels:
-     app.kubernetes.io/instance: synology-csi
-     app.kubernetes.io/managed-by: Helm
-     app.kubernetes.io/name: synology-csi
-     app.kubernetes.io/version: v1.1.2
-     helm.sh/chart: synology-csi-0.9.6-SNAPSHOT
-     helm.sh/template: client-info.yaml
-   name: synology-csi-client-info
- data:
-   client-info.yaml: '-------- # (248 bytes)'

The client-info.yaml file is sill present at synology-csi/templates/client-info.yaml

Can I remove the namespace and reinstall without losing the existing PVCs or is there another solution?

beyondwatts commented 1 year ago

Backed up the cluster and all the pvcs, seemed to be ok with removing the synology-csi namespace and and reinstalling