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

[loki] feat: Add optional prometheusRule value/template #1049

Merged
merged 1 commit into from
Feb 19, 2022

Conversation

eyenx
Copy link
Contributor

@eyenx eyenx commented Feb 17, 2022

Adds an additional prometheusRule template for adding custom Rules which can be used by alertmanager to alert us.

It makes sure prometheusRules are only deployed if serviceMonitor is enabled


Signed-off-by: Toni Tauro toni.tauro@adfinis.com

@eyenx eyenx force-pushed the feat/add-prometheus-rules-for-loki branch 2 times, most recently from 28858ee to d8f750b Compare February 17, 2022 08:31
@zanhsieh zanhsieh changed the title feat(loki): Add optional prometheusRule value/template [loki] feat: Add optional prometheusRule value/template Feb 17, 2022
@zanhsieh
Copy link
Collaborator

@eyenx
You need to bump the chart to the latest version, otherwise lint test won't pass.

@zanhsieh zanhsieh self-requested a review February 18, 2022 01:39
charts/loki/Chart.yaml Outdated Show resolved Hide resolved
Signed-off-by: Toni Tauro <toni.tauro@adfinis.com>
@eyenx eyenx force-pushed the feat/add-prometheus-rules-for-loki branch from b88161c to 3cd2765 Compare February 18, 2022 05:55
@zanhsieh zanhsieh merged commit 0c83430 into grafana:main Feb 19, 2022
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.

None yet

2 participants