Skip to content

Commit

Permalink
fix(helm): Add conditional to include ruler config only when enabled (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
Tostaza authored Dec 13, 2024
1 parent 17bf32b commit 3badbb3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions production/helm/loki/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ Entries should include a reference to the pull request that introduced the chang

[//]: # (<AUTOMATED_UPDATES_LOCATOR> : do not remove this line. This locator is used by the CI pipeline to automatically create a changelog entry for each new Loki release. Add other chart versions and respective changelog entries bellow this line.)

- [BUGFIX] Add conditional to include ruler config only if `ruler.enabled=true`
- [BUGFIX] Disables the Helm test pod when `test.enabled=false`.

## 6.23.0
Expand Down
2 changes: 2 additions & 0 deletions production/helm/loki/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,9 @@ loki:
{{- toYaml .Values.loki.testSchemaConfig | nindent 2}}
{{- end }}
{{- if .Values.ruler.enabled }}
{{ include "loki.rulerConfig" . }}
{{- end }}
{{- if or .Values.tableManager.retention_deletes_enabled .Values.tableManager.retention_period }}
table_manager:
Expand Down

0 comments on commit 3badbb3

Please sign in to comment.