Open reefland opened 2 years ago
Thanks for taking the time to write up this issue. Exposing metrics / serviceMonitor is not a feature of the common library though, so this repo is not the correct place for it. I've moved it over to the charts repo, where it might be better suited.
Details
Helm chart name and version:
Issue when working with mqtt-exporter.
What steps did you take and what happened:
Possible missing section? mqtt-exporter section below indicates to check
@default -- See values.yaml
When I review the common/values.yaml there is nothing about
metrics
orserviceMonitor
in it, seems like there should be.What did you expect to happen:
Expected to see metrics section within the common
values.yaml
Anything else you would like to add:
The mqtt-exporter chart, creates a service for no clear reason I could see, I assume that is for the serviceMonitor. This seems incorrect. a PodMonitor should be used for applications that do not require a service. It's not clear if a PodMonitor is supported since the metrics section is missing.
When I curl against the serviceMonitor, I get data back but there is really nothing about MQTT included:
There is nothing about the broker, clients connected, etc. which I assume there should be?
The exporter is connected and subscribed to the broker: