qdrant-helm icon indicating copy to clipboard operation
qdrant-helm copied to clipboard

ServieMonitor should use with higher precedence the readOnlyApiKey

Open vitali-ipquants opened this issue 1 year ago • 1 comments

Hello Qdrant team,

If in the values.yaml are defined both apiKey and readOnlyApiKey it will be better if the ServiceMonitor refers to the read-only key. The Prometheus will be able to collect metrics with the read-only key, so there's no need to pass it the more powerful key.

Thank you

vitali-ipquants avatar Aug 12 '24 14:08 vitali-ipquants

Good catch! I've create a PR for it here https://github.com/qdrant/qdrant-helm/pull/221

arthurk avatar Aug 13 '24 11:08 arthurk