fix(helm): add temporary storage volume and mount for sidecar rules

Signed-off-by: Jan-Otto Kröpke <[email protected]>

Author: jkroepke

production/helm/loki/CHANGELOG.md

@@ -13,6 +13,7 @@ Entries should include a reference to the pull request that introduced the chang
 
 ## Unreleased
 
+- [BUGFIX] add temporary storage volume and mount for sidecar rule [#19973](https://github.com/grafana/loki/pull/19973)
 - [BUGFIX] Don't mark loki.storage.bucketNames.chunks as required, if an s3 url is provided. [#19873](https://github.com/grafana/loki/pull/19873)
 - [BUGFIX] Do not mark loki.storage.bucketNames.chunks as required, if minio is used. [#19871](https://github.com/grafana/loki/pull/19871)
 - [ENHANCEMENT] Use fsGroupChangePolicy=OnRootMismatch on loki to speed up pod starts [#13942](https://github.com/grafana/loki/pull/13942)

production/helm/loki/templates/backend/statefulset-backend.yaml

@@ -218,6 +218,8 @@ spec:
           volumeMounts:
             - name: sc-rules-volume
               mountPath: {{ .Values.sidecar.rules.folder | quote }}
+            - name: sc-tmp
+              mountPath: /tmp
         {{- end}}
         {{- with .Values.backend.extraContainers }}
         {{- toYaml . | nindent 8}}
@@ -245,10 +247,14 @@ spec:
       volumes:
         - name: tmp
           emptyDir: {}
+        {{- if .Values.sidecar.rules.enabled }}
+        - name: sc-tmp
+          emptyDir: {}
+        {{- end }}
         {{- if not .Values.backend.persistence.volumeClaimsEnabled }}
         - name: data
         {{- toYaml .Values.backend.persistence.dataVolumeParameters | nindent 10 }}
-        {{- end}}
+        {{- end }}
         - name: config
           {{- include "loki.configVolume" . | nindent 10 }}
         - name: runtime-config

production/helm/loki/templates/ruler/statefulset-ruler.yaml

@@ -192,6 +192,8 @@ spec:
           volumeMounts:
             - name: sc-rules-volume
               mountPath: {{ .Values.sidecar.rules.folder | quote }}
+            - name: sc-tmp
+              mountPath: /tmp/
         {{- end}}
         {{- with .Values.ruler.extraContainers }}
         {{- toYaml . | nindent 8}}
@@ -243,6 +245,10 @@ spec:
         {{- end }}
         - name: tmp
           emptyDir: {}
+        {{- if .Values.sidecar.rules.enabled }}
+        - name: sc-tmp
+          emptyDir: {}
+        {{- end }}
         {{- with (concat .Values.global.extraVolumes .Values.ruler.extraVolumes) | uniq }}
         {{- toYaml . | nindent 8 }}
         {{- end }}

production/helm/loki/templates/single-binary/statefulset.yaml

@@ -228,6 +228,8 @@ spec:
           volumeMounts:
             - name: sc-rules-volume
               mountPath: {{ .Values.sidecar.rules.folder | quote }}
+            - name: sc-tmp
+              mountPath: /tmp
         {{- end}}
         {{- with .Values.singleBinary.extraContainers }}
         {{- toYaml . | nindent 8 }}
@@ -251,6 +253,10 @@ spec:
       volumes:
         - name: tmp
           emptyDir: {}
+        {{- if .Values.sidecar.rules.enabled }}
+        - name: sc-tmp
+          emptyDir: {}
+        {{- end }}
         - name: config
           {{- include "loki.configVolume" . | nindent 10 }}
         - name: runtime-config