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.

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

Unconditional rulerConfig in values.yaml #15357

Closed
Tostaza opened this issue Dec 11, 2024 · 1 comment · Fixed by #15385
Closed

Unconditional rulerConfig in values.yaml #15357

Tostaza opened this issue Dec 11, 2024 · 1 comment · Fixed by #15385

Comments

@Tostaza
Copy link
Contributor

Tostaza commented Dec 11, 2024

Is your feature request related to a problem? Please describe.

I've been recently trying to install loki and configure it to store logs in an external minio instance. I'm not using the ruler component since I only want to store chunks in one bucket. Even though I have disabled the ruler in my values.yaml, the config section does not check it and unconditionally includes a rulerConfiguration. Since I have not provided a bucket for the ruler, the loki-backend pod fails and enters CrashLoopBackOff state

Describe the solution you'd like
Add an if statement in the default values.yaml to include loki.rulerConfig only if ruler.enabled: true

Describe alternatives you've considered
If anybody has the same problem, some temporal fixes would be:

  • Add the if statement in the provided custom values. This means you should copy all the config section in your values.yaml.
  • You can also specify an existing bucket for the ruler (e.g the same as for chunks). This will avoid the loki-backend failing. Notice that even if its specified, the ruler will not store info since its disabled

{{ include "loki.rulerConfig" . }}

@Tostaza
Copy link
Contributor Author

Tostaza commented Dec 12, 2024

A solution is provided here: #15385

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 a pull request may close this issue.

1 participant