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

Jump to bottom

feat(helm): Add commonly used configurations to global scope #14406

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

feat(helm): Add commonly used configurations to global scope #14406

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9672e25
Add: extraEnv, extraEnvFrom init
RotemCDos Sep 30, 2024
93405e5
Add: extraEnv, extraEnvFrom tokengen
RotemCDos Oct 1, 2024
76c5ef3
Fix: extraEnv, extraEnvFrom to loki scope
RotemCDos Oct 1, 2024
247a4e1
Fix: extraEnv, extraEnvFrom to loki scope
RotemCDos Oct 1, 2024
afaafbf
Fix: align extraArgs to list
RotemCDos Oct 6, 2024
9df60f5
Feat: global extraArgs
RotemCDos Oct 6, 2024
2b85aba
Fix: memcached-sts ctx
RotemCDos Oct 6, 2024
43fd481
fix: memcache-sts extraArgs tpl
RotemCDos Oct 6, 2024
6ae2977
fix: memcache-sts extraArgs tpl
RotemCDos Oct 6, 2024
d917a4c
fix: loki-canary daemonset extraArgs
RotemCDos Oct 6, 2024
bd6e7af
Add: templated tpl functions for new global configs
RotemCDos Oct 6, 2024
fc4a726
Feat: templating functions with usage
RotemCDos Oct 7, 2024
e47f75f
Revert: extraArgs to dict
RotemCDos Oct 7, 2024
418bc4b
Fix: typos
RotemCDos Oct 7, 2024
83a4a7a
Fix: missing enterprise in tokengen
RotemCDos Oct 7, 2024
e426802
Merge pull request #5 from grafana/main
RotemCDos Oct 7, 2024
24fd02b
Fix: memcached extraArgs loop
RotemCDos Oct 7, 2024
2c84ed7
Merge branch 'feat/global-config' of https://github.com/Observabilite…
RotemCDos Oct 7, 2024
b4a46cf
Merge branch 'main' into feat/global-config
ohad2b Nov 11, 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
30 changes: 30 additions & 0 deletions production/helm/loki/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1121,3 +1121,33 @@ Return the appropriate apiVersion for HorizontalPodAutoscaler.
{{- print "autoscaling/v2beta1" -}}
{{- end -}}
{{- end -}}

{{/*
Return the templated list for extraEnv and extraEnvFrom
Params:
. = extraEnv/extraEnvFrom list
*/}}
{{- define "loki.templateEnv" -}}
{{- $extraEnv := . }}
{{- with $extraEnv }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{- end -}}

{{/*
Return the templated dict/list for extraArgs
Params:
. = extraArgs dict/list
*/}}
{{- define "loki.templateArgs" -}}
{{- $extraArgs := . }}
{{- if and $extraArgs (kindIs "slice" $extraArgs) }}
{{- with $extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{- else if and $extraArgs (kindIs "map" $extraArgs) -}}
{{- range $key, $value := $extraArgs }}
- "-{{ $key }}={{ $value }}"
{{- end -}}
{{- end -}}
{{- end -}}
17 changes: 7 additions & 10 deletions production/helm/loki/templates/admin-api/deployment-admin-api.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,9 +88,8 @@ spec:
- -admin.client.s3.secret-access-key={{ .Values.minio.secretKey }}
- -admin.client.s3.insecure=true
{{- end }}
{{- range $key, $value := .Values.adminApi.extraArgs }}
- "-{{ $key }}={{ $value }}"
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.adminApi.extraArgs }}
volumeMounts:
- name: config
mountPath: /etc/loki/config
Expand Down Expand Up @@ -118,13 +117,11 @@ spec:
securityContext:
{{- toYaml .Values.adminApi.containerSecurityContext | nindent 12 }}
env:
{{- if .Values.adminApi.env }}
{{ toYaml .Values.adminApi.env | nindent 12 }}
{{- end }}
{{- with .Values.adminApi.extraEnvFrom }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.adminApi.extraEnv }}
envFrom:
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.adminApi.extraEnvFrom }}
{{- with .Values.adminApi.extraContainers }}
{{ toYaml . | nindent 8 }}
{{- end }}
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/backend/statefulset-backend.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -160,9 +160,8 @@ spec:
- -config.file=/etc/loki/config/config.yaml
- -target={{ .Values.backend.targetModule }}
- -legacy-read-mode=false
{{- with .Values.backend.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.backend.extraArgs }}
ports:
- name: http-metrics
containerPort: {{ .Values.loki.server.http_listen_port }}
Expand All @@ -173,14 +172,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.backend.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.backend.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.backend.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.backend.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
readinessProbe:
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/bloom-builder/deployment-bloom-builder.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,8 @@ spec:
args:
- -config.file=/etc/loki/config/config.yaml
- -target=bloom-builder
{{- with .Values.bloomBuilder.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.bloomBuilder.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -80,14 +79,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.bloomBuilder.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.bloomBuilder.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.bloomBuilder.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.bloomBuilder.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
readinessProbe:
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/bloom-gateway/statefulset-bloom-gateway.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,8 @@ spec:
args:
- -config.file=/etc/loki/config/config.yaml
- -target=bloom-gateway
{{- with .Values.bloomGateway.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.bloomGateway.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -88,14 +87,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.bloomGateway.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.bloomGateway.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.bloomGateway.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.bloomGateway.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
{{- include "loki.bloomGateway.readinessProbe" . | nindent 10 }}
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/bloom-planner/statefulset-bloom-planner.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,8 @@ spec:
args:
- -config.file=/etc/loki/config/config.yaml
- -target=bloom-planner
{{- with .Values.bloomPlanner.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.bloomPlanner.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -88,14 +87,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.bloomPlanner.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.bloomPlanner.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.bloomPlanner.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.bloomPlanner.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
{{- include "loki.bloomPlanner.readinessProbe" . | nindent 10 }}
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/compactor/statefulset-compactor.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,8 @@ spec:
args:
- -config.file=/etc/loki/config/config.yaml
- -target=compactor
{{- with .Values.compactor.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.compactor.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -95,14 +94,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.compactor.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.compactor.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.compactor.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.compactor.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
{{- include "loki.compactor.readinessProbe" . | nindent 10 }}
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/distributor/deployment-distributor.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,8 @@ spec:
- -distributor.zone-awareness-enabled=true
{{- end }}
{{- end }}
{{- with .Values.distributor.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.distributor.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -88,14 +87,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.distributor.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.distributor.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.distributor.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.distributor.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
readinessProbe:
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/gateway/deployment-gateway-enterprise.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,9 +87,8 @@ spec:
- -gateway.proxy.ruler.url=http://{{ template "loki.backendFullname" . }}-headless.{{ .Release.Namespace }}.svc:3100
- -gateway.proxy.query-scheduler.url=http://{{ template "loki.backendFullname" . }}-headless.{{ .Release.Namespace }}.svc:3100
{{- end }}
{{- range $key, $value := .Values.enterpriseGateway.extraArgs }}
- "-{{ $key }}={{ $value }}"
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.enterpriseGateway.extraArgs }}
volumeMounts:
- name: config
mountPath: /etc/loki/config
Expand All @@ -111,13 +110,11 @@ spec:
securityContext:
{{- toYaml .Values.enterpriseGateway.containerSecurityContext | nindent 12 }}
env:
{{- if .Values.enterpriseGateway.env }}
{{ toYaml .Values.enterpriseGateway.env | nindent 12 }}
{{- end }}
{{- with .Values.enterpriseGateway.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.enterpriseGateway.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.enterpriseGateway.extraEnvFrom }}
{{- with .Values.enterpriseGateway.extraContainers }}
{{ toYaml . | nindent 8 }}
{{- end }}
Expand Down
8 changes: 2 additions & 6 deletions production/helm/loki/templates/gateway/deployment-gateway-nginx.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,14 +64,10 @@ spec:
- name: http-metrics
containerPort: {{ .Values.gateway.containerPort }}
protocol: TCP
{{- with .Values.gateway.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.gateway.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.gateway.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.gateway.extraEnvFrom }}
readinessProbe:
{{- toYaml .Values.gateway.readinessProbe | nindent 12 }}
securityContext:
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/index-gateway/statefulset-index-gateway.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,9 +76,8 @@ spec:
args:
- -config.file=/etc/loki/config/config.yaml
- -target=index-gateway
{{- with .Values.indexGateway.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.indexGateway.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -91,14 +90,12 @@ spec:
containerPort: 7946
protocol: TCP
{{- end }}
{{- with .Values.indexGateway.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.indexGateway.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.indexGateway.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.indexGateway.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
readinessProbe:
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/ingester/statefulset-ingester-zone-a.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,8 @@ spec:
- -ingester.unregister-on-shutdown=false
- -ingester.tokens-file-path=/var/loki/ring-tokens
- -target=ingester
{{- with .Values.ingester.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.ingester.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -119,14 +118,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.ingester.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.ingester.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.ingester.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.ingester.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
{{- include "loki.ingester.readinessProbe" . | nindent 10 }}
Expand Down
15 changes: 6 additions & 9 deletions production/helm/loki/templates/ingester/statefulset-ingester-zone-b.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,8 @@ spec:
- -ingester.unregister-on-shutdown=false
- -ingester.tokens-file-path=/var/loki/ring-tokens
- -target=ingester
{{- with .Values.ingester.extraArgs }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateArgs" .Values.loki.extraArgs }}
{{ include "loki.templateArgs" .Values.ingester.extraArgs }}
ports:
- name: http-metrics
containerPort: 3100
Expand All @@ -119,14 +118,12 @@ spec:
- name: http-memberlist
containerPort: 7946
protocol: TCP
{{- with .Values.ingester.extraEnv }}
env:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- with .Values.ingester.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.loki.extraEnv }}
{{ include "loki.templateEnv" .Values.ingester.extraEnv }}
envFrom:
{{- toYaml . | nindent 12 }}
{{- end }}
{{ include "loki.templateEnv" .Values.loki.extraEnvFrom }}
{{ include "loki.templateEnv" .Values.ingester.extraEnvFrom }}
securityContext:
{{- toYaml .Values.loki.containerSecurityContext | nindent 12 }}
{{- include "loki.ingester.readinessProbe" . | nindent 10 }}
Expand Down
Loading
Loading