Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(helm): only default bucket names when using minio #12548

Merged
merged 8 commits into from
Apr 9, 2024
Merged

fix(helm): only default bucket names when using minio #12548

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3667d2a
fix(helm): only default bucket names when using minio
trevorwhitney Apr 9, 2024
32ec9be
chore: bring back bucket config with FIXMEs
trevorwhitney Apr 9, 2024
85e987e
fix: docs
trevorwhitney Apr 9, 2024
b53f0fa
Merge branch 'main' into mo-buckets-mo-problems
trevorwhitney Apr 9, 2024
a726172
ci: fix default-single-binary-values
trevorwhitney Apr 9, 2024
7fa7127
ci: fix a few more values
trevorwhitney Apr 9, 2024
0c5fed1
chore: bump helm version
trevorwhitney Apr 9, 2024
2f1c44d
chore: add helm-docs make target
trevorwhitney Apr 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions production/helm/loki/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ Generated storage config for loki common config
{{- if .Values.minio.enabled -}}
s3:
endpoint: {{ include "loki.minio" $ }}
bucketnames: {{ $.Values.loki.storage.bucketNames.chunks }}
bucketnames: chunks
secret_access_key: {{ $.Values.minio.rootPassword }}
access_key_id: {{ $.Values.minio.rootUser }}
s3forcepathstyle: true
Expand Down Expand Up @@ -346,7 +346,7 @@ Storage config for ruler
{{- if .Values.minio.enabled -}}
type: "s3"
s3:
bucketnames: {{ $.Values.loki.storage.bucketNames.ruler }}
bucketnames: ruler
{{- else if eq .Values.loki.storage.type "s3" -}}
{{- with .Values.loki.storage.s3 }}
type: "s3"
Expand Down
6 changes: 1 addition & 5 deletions production/helm/loki/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -311,10 +311,6 @@ loki:
compactor_address: '{{ include "loki.compactorAddress" . }}'
# -- Storage config. Providing this will automatically populate all necessary storage configs in the templated config.
storage:
bucketNames:
chunks: chunks
ruler: ruler
admin: admin
Comment on lines -314 to -317
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of deleting this, maybe comment it out with FIXME for the names?

type: s3
s3:
s3: null
Expand Down Expand Up @@ -477,7 +473,7 @@ enterprise:
admin_client:
storage:
s3:
bucket_name: {{ .Values.loki.storage.bucketNames.admin }}
bucket_name: admin
{{- end }}
{{- end }}
auth:
Expand Down
Loading