ceph / ceph-helm

Curated applications for Kubernetes
Apache License 2.0
108 stars 36 forks source link

Integration of Ceph Exporter #44

Closed betermieux closed 6 years ago

betermieux commented 6 years ago

Hi, I was looking for a metrics exporter for prometheus and found a general exporter at: ceph/ceph-container/examples/helm/ceph

Would it be possible to integrate the exporter into this project? I am not very good at writing helm charts, and the structure of both charts seems to differ a lot.

Thanks, Stefan

rootfs commented 6 years ago

cc @dmick

dmick commented 6 years ago

The strategy for Ceph and Prometheus is an exporter built as a manager module. Enabling the module and advertising the address/port should be done by the manager deployment. I'm not certain if that's yet done, but that's where to look.

dmick commented 6 years ago

By manager I mean ceph-mgr

betermieux commented 6 years ago

Yes, the manager module prometheus looks appropriate. It is working out of the box by adding the following snippet to ceph-overrides.yaml:

ceph_mgr_enabled_modules:
  - restful
  - status
  - prometheus

There seems to be no chance to add metadata/annotations to the deployment of the manager. I helped myself by patching the deployment afterwards.

dmick commented 6 years ago

Easy enough to add Values for annotations