Skip to content

helm: add PodMonitor support to ceph-csi-rbd Helm chart#6287

Open
nepomucen wants to merge 1 commit into
ceph:develfrom
nepomucen:add/prom-cr
Open

helm: add PodMonitor support to ceph-csi-rbd Helm chart#6287
nepomucen wants to merge 1 commit into
ceph:develfrom
nepomucen:add/prom-cr

Conversation

@nepomucen

Copy link
Copy Markdown

Describe what this PR does

This pull request adds support for monitoring the provisioner component of the Ceph-CSI RBD Helm chart via Prometheus by introducing an optional PodMonitor resource.
This enables users to easily integrate metrics collection for the provisioner and its sidecars when Prometheus Operator is installed.

provisioner.podMonitor.enabled=false

Checklist:

  • Commit Message Formatting: Commit titles and messages follow
    guidelines in the developer
    guide
    .
  • Reviewed the developer guide on Submitting a Pull
    Request
  • Pending release
    notes

    updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

Signed-off-by: nepomucen <szymon@aus.krakow.pl>
@mergify mergify Bot added the component/deployment Helm chart, kubernetes templates and configuration Issues/PRs label May 19, 2026
@nepomucen

Copy link
Copy Markdown
Author

@jd would you be so kind and review my PR. This is something we have been patching ever since with strategicMergePatches in our helm release. Would be very nice to have it included in the official helm chart.

@jd

jd commented May 27, 2026

Copy link
Copy Markdown
Contributor

@jd would you be so kind and review my PR. This is something we have been patching ever since with strategicMergePatches in our helm release. Would be very nice to have it included in the official helm chart.

Not really my thing, sorry.

@nixpanic

Copy link
Copy Markdown
Member

This is quite nice and simple. But the Helm Charts are being deprecated (slowly), and replaced by Ceph-CSI Operator. It probably makes sense to add (optional) support for PodMonitor to Ceph-CSI Operator instead.

Is there a good reason to only add it for provisioners, and not the node-plugins? PodMonitor should be added to other storage types too.

@nixpanic nixpanic added the question Further information is requested label Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/deployment Helm chart, kubernetes templates and configuration Issues/PRs question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants