sagikazarmark / helm-charts

Various Helm charts for my own and other projects
MIT License
43 stars 55 forks source link

Remove suffix -pvc in the end of persistentVolumeClaim name in deployment #220

Open aeciopires opened 2 months ago

aeciopires commented 2 months ago

Hi guys!

Thanks for great work in sftp-go and this helm chart.

I found a problem deploying this helm chart in GKE Standard using persistent volume.

Error message:

0/1 nodes are available: 1 pod has unbound immediate PersistentVolumeClaims

In templates/deployment.yaml file

As is:

[...]
      volumes:
        [...]
         - name: sftpgo-volume
           persistentVolumeClaim:
             claimName: {{ include "sftpgo.fullname" . }}-pvc
[...]

Solution suggested:

[...]
      volumes:
        [...]
         - name: sftpgo-volume
           persistentVolumeClaim:
             claimName: {{ include "sftpgo.fullname" . }}
[...]

I solved the issue by editing the deployment manually, but this issue needs to be fixed in a new version of the helm chart.

Lunatunny commented 2 months ago

Yuppers same issue.