Is your feature request related to a problem or existing issue? Please describe.
Currently, the Helm chart exposes metrics via a Service but it lacks a template for a ServiceMonitor CRD.
For users running the Prometheus, they are forced to manually write & maintain their own ServiceMonitor manifests outside of the Helm release.
While users can technically use standard prometheus.io/scrape pod annotations, those annotations are inadequate for scraping secure HTTPS endpoints since they cannot natively pass tlsConfig, caFile, or bearerToken to Prometheus.
Describe the solution you'd like
Adding a conditionally rendered ServiceMonitor template to the Helm chart to provide native Prometheus Operator integration.
This will be disabled by default to avoid breaking clusters without the operator installed.
Describe alternatives you've considered
It will brings NRC up to parity with standard CNCF controllers
Is your feature request related to a problem or existing issue? Please describe.
Currently, the Helm chart exposes metrics via a Service but it lacks a template for a ServiceMonitor CRD.
For users running the Prometheus, they are forced to manually write & maintain their own ServiceMonitor manifests outside of the Helm release.
While users can technically use standard prometheus.io/scrape pod annotations, those annotations are inadequate for scraping secure HTTPS endpoints since they cannot natively pass tlsConfig, caFile, or bearerToken to Prometheus.
Describe the solution you'd like
Adding a conditionally rendered ServiceMonitor template to the Helm chart to provide native Prometheus Operator integration.
This will be disabled by default to avoid breaking clusters without the operator installed.
Describe alternatives you've considered
It will brings NRC up to parity with standard CNCF controllers