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

[Helm Chart Loki] Add needed k8s objects for alerting config #2965

Merged
merged 1 commit into from
Nov 23, 2020

Conversation

czunker
Copy link
Contributor

@czunker czunker commented Nov 20, 2020

  • ConfigMap containing the rules
  • VolumeMount for rules
  • scratch mount for rule parsing

Fixes #2964

Signed-off-by: Christian Zunker christian.zunker@codecentric.cloud

Checklist

  • Documentation added
  • [-] Tests updated

@CLAassistant
Copy link

CLAassistant commented Nov 20, 2020

CLA assistant check
All committers have signed the CLA.

@codecov-io
Copy link

Codecov Report

Merging #2965 (f00ac44) into master (a62f81f) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2965      +/-   ##
==========================================
- Coverage   61.75%   61.74%   -0.02%     
==========================================
  Files         182      182              
  Lines       14866    14866              
==========================================
- Hits         9181     9179       -2     
- Misses       4838     4840       +2     
  Partials      847      847              
Impacted Files Coverage Δ
pkg/querier/queryrange/downstreamer.go 95.29% <0.00%> (-2.36%) ⬇️

@czunker
Copy link
Contributor Author

czunker commented Nov 20, 2020

CI failed on cloning fluent-bit-amd64. So, nothing I can fix. Is there a way to recheck?

- ConfigMap containing the rules
- VolumeMount for rules
- scratch mount for rule parsing

Fixes grafana#2964

Signed-off-by: Christian Zunker <christian.zunker@codecentric.cloud>
Copy link
Contributor

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the addition.

@cyriltovena cyriltovena merged commit 57f8159 into grafana:master Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Helm Chart Loki] add needed Objects for Loki Alerting
4 participants