fix(helm): do not mark loki.storage.bucketNames.chunks as required, if minio is used.

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] Do not mark loki.storage.bucketNames.chunks as required, if minio is used. [#19871](https://github.com/grafana/loki/pull/19871)
 
 ## 6.46.0
 

production/helm/loki/ci/non-default-values.yaml

@@ -6,10 +6,6 @@ loki:
   commonConfig:
     replication_factor: 1
   useTestSchema: true
-  storage:
-    bucketNames:
-      chunks: chunks
-      ruler: ruler
   tenants:
     - name: "test-user-1"
       password: "test-password-1"

production/helm/loki/templates/_helpers.tpl

@@ -208,7 +208,6 @@ Generated storage config for loki common config
 object_store:
   {{- include "loki.thanosStorageConfig" (dict "ctx" . "bucketName" $bucketName) | nindent 2 }}
 {{- else if .Values.minio.enabled -}}
-{{- $bucketName := required "Please define loki.storage.bucketNames.chunks" (dig "storage" "bucketNames" "chunks" "" .Values.loki) }}
 s3:
   endpoint: {{ include "loki.minio" $ }}
   bucketnames: chunks