This is an automated email from the ASF dual-hosted git repository.

gerlowskija pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr-operator.git


The following commit(s) were added to refs/heads/main by this push:
     new ccf54fd  Correct query-param name used by prom-exporter probe (#729)
ccf54fd is described below

commit ccf54fd61dc593f38086e79c6c985118ffb1ece8
Author: Steffen Moldenhauer <54577793+smoldenhauer-...@users.noreply.github.com>
AuthorDate: Mon Dec 9 17:06:16 2024 +0100

    Correct query-param name used by prom-exporter probe (#729)
    
    A previous change introduced the use of the `names` parameter to avoid
    exporting metrics, but the correct param-name is `name` (no '-s' suffix).
---
 controllers/util/prometheus_exporter_util.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/controllers/util/prometheus_exporter_util.go 
b/controllers/util/prometheus_exporter_util.go
index fd8f9a2..1792e88 100644
--- a/controllers/util/prometheus_exporter_util.go
+++ b/controllers/util/prometheus_exporter_util.go
@@ -208,7 +208,7 @@ func 
GenerateSolrPrometheusExporterDeployment(solrPrometheusExporter *solr.SolrP
                HTTPGet: &corev1.HTTPGetAction{
                        Scheme: corev1.URISchemeHTTP,
                        // TODO: When 9.0 is the minimum supported version, 
this can be "/-/healthy"
-                       Path: "/metrics?names[]=",
+                       Path: "/metrics?name[]=",
                        Port: intstr.FromInt(SolrMetricsPort),
                },
        }

Reply via email to