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

feat: add Grafana dashboards and alerts to monitor Thanos #73

Merged
merged 3 commits into from
Feb 9, 2024

Conversation

lentidas
Copy link
Contributor

@lentidas lentidas commented Feb 9, 2024

Description of the changes

This PR adds multiple Grafana dashboards and Prometheus alerts to monitor Thanos.

The alerts are conditionally deployed depending on the Thanos components deployed. Note that the sidecar rules are always deployed, because the DevOps Stack was designed for Thanos in the sidecar mode and not in the receiver mode. When we change this implementation, we should be aware of that.

Also, for the time being, the dashboards are always deployed, even if certain components are not. Note that certain dashboard files are suffixed with .ignore to disable them. I'll let this be done statically until I figure out a way to do it while still using the iteration over the files in the grafana-dashboard.yaml template.

⚠️ Do a Rebase and merge

Breaking change

  • No

Tests executed on which distribution(s)

  • AKS (Azure)
  • EKS (AWS)
  • SKS (Exoscale)

jbarascut
jbarascut previously approved these changes Feb 9, 2024
@lentidas lentidas merged commit 68134ae into main Feb 9, 2024
@lentidas lentidas deleted the ISDEVOPS-254 branch February 9, 2024 12:28
@lentidas lentidas mentioned this pull request Feb 21, 2024
8 tasks
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