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

Grafana #257

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Grafana #257

wants to merge 8 commits into from

Conversation

BuJo
Copy link
Member

@BuJo BuJo commented Sep 1, 2024

Add Grafana connector showing alerts

@BuJo BuJo self-assigned this Sep 1, 2024
BuJo added 2 commits September 1, 2024 14:33
* Note that grafana api is "lying".  The published alerts are not firing and
  are still published as alerting.
* This behaviour mirrors the intent of the provisioning api, which also
  hides the current alert state.
* the prometheus v1 API rules at least provides the current state, as
  opposed to:
  * /api/ruler/grafana/api/v1/rules/
  * /api/alertmanager/grafana/api/v2/alerts
  * /api/prometheus/grafana/api/v1/alerts
  * /api/v1/provisioning/alert-rules
  * /api/alerts
* This reverts commit a2214be.
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.

1 participant