[kube-prometheus-stack] ServiceMonitor needs separate scheme parameters

[tynsh]

Hi,

currently the prometheus config reloader does not support https at the metrics endpoint. As both the config reloader and prometheus/alertmanager share the same scheme value, when activating https on the serviceMonitor scraping of will fail on the reloader container. This issue exists for the prometheus service monitor as well as the one for alertmanager. Here are the relevant lines for alertmanager:

https://github.com/prometheus-community/helm-charts/blob/723cfad365d5c55b84162d4a5f11294ea736ca54/charts/kube-
prometheus-stack/templates/alertmanager/servicemonitor.yaml#L57-L59

[n1kofr]

Facing the same issue for prometheus config reloader. Scrapping config-reloader is failing with wrong scheme issue as config-reloader ServiceMonitor is using the same scheme as prometheus.

helm-charts/charts/kube-prometheus-stack/templates/prometheus/servicemonitor.yaml

Lines 50 to 52 in 723cfad

	{{- if .Values.prometheus.serviceMonitor.scheme }}
	scheme: {{ .Values.prometheus.serviceMonitor.scheme }}
	{{- end }}