memgraph / helm-charts

Helm charts for deploying Memgraph, an open-source in-memory graph database.
https://memgraph.github.io/helm-charts/
Apache License 2.0
14 stars 11 forks source link

[Bug]: Volume mount name for log storage is incorrect #60

Closed dintorf closed 2 weeks ago

dintorf commented 2 months ago

Contact Details

No response

What happened?

The volume mount name for the log storage claim is incorrect. It points to the lib storage claim.

https://github.com/memgraph/helm-charts/blob/main/charts/memgraph/templates/statefulset.yaml#L181-L184

          {{- if .Values.persistentVolumeClaim.createLogStorage }}
            - name: {{ include "memgraph.fullname" . }}-lib-storage # here is the incorrect line
              mountPath: /var/log/memgraph
          {{- end }}

Should be changed to

          {{- if .Values.persistentVolumeClaim.createLogStorage }}
            - name: {{ include "memgraph.fullname" . }}-log-storage # should be log-storage here instead of lib-storage
              mountPath: /var/log/memgraph
          {{- end }}

Chart type

Standalone

Chart version

0.1.x

Environment

My environment is not listed

Relevant log output

No response

antejavor commented 2 months ago

Hi @dintorf, thanks for letting me know about this, we will fix it! 🚀