Skip to content

feat: add stacklight ceph alert rules and dashboard#879

Merged
gmlexx merged 3 commits into
mainfrom
ceph-alerts
Apr 14, 2026
Merged

feat: add stacklight ceph alert rules and dashboard#879
gmlexx merged 3 commits into
mainfrom
ceph-alerts

Conversation

@gmlexx
Copy link
Copy Markdown
Collaborator

@gmlexx gmlexx commented Mar 24, 2026

Closes #861

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds Ceph observability artifacts to the Helm charts by enabling a new Ceph default-rules group in the mothership Prometheus rules and introducing a Ceph Grafana dashboard in the dashboards chart.

Changes:

  • Enable defaultRules.rules.ceph and add per-group label/annotation override stubs in kof-mothership values.
  • Add a new PrometheusRule manifest (ceph.yaml) containing Ceph alerting rules gated by defaultRules.rules.ceph.
  • Add a new Ceph Grafana dashboard definition under kof-dashboards.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.

File Description
charts/kof-mothership/values.yaml Adds ceph toggles/override maps to wire the new default rules group into existing configuration.
charts/kof-mothership/templates/prometheus/rules/ceph.yaml Introduces a new Ceph PrometheusRule group controlled by Helm values.
charts/kof-dashboards/files/dashboards/ceph/ceph.yaml Adds a Ceph dashboard (Grafana model in YAML) for cluster/OSD/PG/latency views.

Comment thread charts/kof-dashboards/files/dashboards/ceph/ceph.yaml
Comment thread charts/kof-dashboards/files/dashboards/ceph/ceph.yaml Outdated
Comment thread charts/kof-dashboards/files/dashboards/ceph/ceph.yaml Outdated
Comment thread charts/kof-dashboards/files/dashboards/ceph/ceph.yaml
Comment thread charts/kof-mothership/templates/prometheus/rules/ceph.yaml
@gmlexx gmlexx force-pushed the ceph-alerts branch 4 times, most recently from e3d5a69 to 1a5d977 Compare April 8, 2026 08:40
gmlexx and others added 3 commits April 13, 2026 13:57
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@gmlexx gmlexx merged commit 1f062b8 into main Apr 14, 2026
28 of 38 checks passed
@gmlexx gmlexx deleted the ceph-alerts branch April 14, 2026 14:56
@github-project-automation github-project-automation Bot moved this to Done in k0rdent Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Ceph Monitoring Capability

5 participants