Skip to content

[loki] Fix loki ruler directories mount#323

Merged
jkroepke merged 11 commits intografana-community:mainfrom
vquie:loki-ruler-directories-mount
Apr 10, 2026
Merged

[loki] Fix loki ruler directories mount#323
jkroepke merged 11 commits intografana-community:mainfrom
vquie:loki-ruler-directories-mount

Conversation

@vquie
Copy link
Copy Markdown

@vquie vquie commented Apr 10, 2026

What this PR does / why we need it

The ruler.directories where not mounted even though loki.rulerConfig.storage was defined.

This PR fixes this and recreates previous behavior.

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [grafana])

vquie added 2 commits April 10, 2026 15:04
Render configMap-backed rule volumes for ruler when
  sidecar.rules.enabled is false
Ensure sidecar volumes and mounts are skipped if disabled
Add tests validating ruler directories without sidecar

Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
Updates Helm chart version for latest Loki release

Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
vquie added 2 commits April 10, 2026 15:09
Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
@vquie vquie force-pushed the loki-ruler-directories-mount branch from eed6a44 to d9eb782 Compare April 10, 2026 13:12
vquie and others added 6 commits April 10, 2026 19:17
Co-authored-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: VQuie <21147643+vquie@users.noreply.github.com>
Co-authored-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: VQuie <21147643+vquie@users.noreply.github.com>
Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
Corrected has usage to pass $target as first argument
Ensures proper evaluation for ruler, backend, single-binary targets

Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
@jkroepke jkroepke self-requested a review April 10, 2026 19:30
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Copy link
Copy Markdown
Member

@jkroepke jkroepke left a comment

Choose a reason for hiding this comment

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

LGTM

@jkroepke jkroepke merged commit 152b414 into grafana-community:main Apr 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants