Skip to content

Commit 0a7f701

Browse files
committed
fix(helm): do not mark loki.storage.bucketNames.ruler as required, if rulerConfig.storage.type is local
Signed-off-by: Jan-Otto Kröpke <[email protected]>
1 parent 019d6b4 commit 0a7f701

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

production/helm/loki/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ Entries should include a reference to the pull request that introduced the chang
1313

1414
## Unreleased
1515

16+
- [BUGFIX] do not mark loki.storage.bucketNames.ruler as required, if rulerConfig.storage.type is local. [#19882](https://github.com/grafana/loki/pull/19882)
1617

1718
## 6.46.0
1819

production/helm/loki/templates/_helpers.tpl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,9 @@ filesystem:
229229
Storage config for ruler
230230
*/}}
231231
{{- define "loki.rulerStorageConfig" -}}
232-
{{- if .Values.minio.enabled -}}
232+
{{- if eq .Values.loki.rulerConfig.storage.type "local" -}}
233+
type: "local"
234+
{{- else if .Values.minio.enabled -}}
233235
type: "s3"
234236
s3:
235237
bucketnames: ruler

0 commit comments

Comments
 (0)