Browse Source

fix: Invalid volume mount conditional in filer template (#7992)

There is a mistmatch in the conditionals for the definition and mounting of the `config-users` volume in the filer's template. 

Volume definition:
```
        {{- if and .Values.filer.s3.enabled .Values.filer.s3.enableAuth }}
```
Mount:
```
            {{- if .Values.filer.s3.enableAuth }}
```

This leads to an invalid specification in the case where s3 is disabled but the enableAuth value is set to true, as it tries to mount in an undefined volume. I've fixed it here by adding the extra check to the latter conditional.
pull/8000/head
Nicholas Boyd Isacsson 3 days ago
committed by GitHub
parent
commit
88e9e2c471
No known key found for this signature in database GPG Key ID: B5690EEEBB952194
  1. 2
      k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml

2
k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml

@ -229,7 +229,7 @@ spec:
- name: seaweedfs-filer-log-volume
mountPath: "/logs/"
{{- end }}
{{- if .Values.filer.s3.enableAuth }}
{{- if and .Values.filer.s3.enabled .Values.filer.s3.enableAuth }}
- name: config-users
mountPath: /etc/sw
readOnly: true

Loading…
Cancel
Save