Skip to content

kolla: Drop the Ceph mgr scrape overlay - #2975

Merged
berendt merged 1 commit into
mainfrom
drop-ceph-scrape-overlay
Oct 16, 2026
Merged

berendt merged 1 commit into
mainfrom
drop-ceph-scrape-overlay

Conversation

@ideaship

Copy link
Copy Markdown
Contributor

environments/kolla/files/overlays/prometheus/prometheus.yml.d/50-ceph.yml adds a ceph scrape job for the Ceph mgr prometheus module. osism/defaults now enables kolla-ansible's own ceph_mgr_exporter job whenever OSISM deploys Ceph and Prometheus is on. With both in place, Prometheus scrapes every Ceph series twice under two job names.

This removes the overlay and its ansible-lint exclusion.

🤖 Generated with Claude Code

prometheus.yml.d/50-ceph.yml adds a "ceph" scrape job for the Ceph mgr
prometheus module, with one target per host in the mons group at its
storage_interface address. osism/defaults now enables kolla-ansible's
own ceph_mgr_exporter job whenever OSISM deploys Ceph and Prometheus
is on, with every ceph-mgr host as a target (osism/defaults#330). With
both in place, Prometheus scrapes every Ceph series twice under two
job names.

Remove the overlay and its ansible-lint exclusion, which existed only
because the file is a Jinja template.

This depends on osism/defaults#330. Until a release carries it,
deployments of that release lose the Ceph scrape.

Assisted-by: Claude:claude-opus-5-5
Signed-off-by: Roger Luethi <luethi@osism.tech>
@ideaship ideaship self-assigned this Oct 16, 2026
@ideaship
ideaship marked this pull request as ready for review October 16, 2026 16:17
@ideaship
ideaship requested a review from berendt October 16, 2026 16:17
@berendt
berendt merged commit 8f3ed99 into main Oct 16, 2026
3 checks passed
@berendt
berendt deleted the drop-ceph-scrape-overlay branch October 16, 2026 16:28
@github-project-automation github-project-automation Bot moved this from New to Done in Human Board Oct 16, 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.

3 participants