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

houston 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 7f2b10c  Remove port annotation form prometheusexporter service (#539)
7f2b10c is described below

commit 7f2b10c2fed2f21d0701277e80ede34333bdf89d
Author: Houston Putman <[email protected]>
AuthorDate: Thu Apr 6 16:45:54 2023 -0400

    Remove port annotation form prometheusexporter service (#539)
---
 controllers/util/prometheus_exporter_util.go | 1 -
 helm/solr-operator/Chart.yaml                | 7 +++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/controllers/util/prometheus_exporter_util.go 
b/controllers/util/prometheus_exporter_util.go
index fd388c4..de70283 100644
--- a/controllers/util/prometheus_exporter_util.go
+++ b/controllers/util/prometheus_exporter_util.go
@@ -406,7 +406,6 @@ func GenerateSolrMetricsService(solrPrometheusExporter 
*solr.SolrPrometheusExpor
                "prometheus.io/scrape": "true",
                "prometheus.io/scheme": "http",
                "prometheus.io/path":   "/metrics",
-               "prometheus.io/port":   strconv.Itoa(ExtSolrMetricsPort),
        }
 
        selectorLabels := solrPrometheusExporter.SharedLabels()
diff --git a/helm/solr-operator/Chart.yaml b/helm/solr-operator/Chart.yaml
index 84cadd2..419e588 100644
--- a/helm/solr-operator/Chart.yaml
+++ b/helm/solr-operator/Chart.yaml
@@ -125,6 +125,13 @@ annotations:
           url: https://github.com/apache/solr-operator/pull/530
         - name: Feature Documentation
           url: 
https://apache.github.io/solr-operator/docs/solr-cloud/managed-updates.html#pod-readiness-during-updates
+    - kind: fixed
+      description: Don't give port annotation for PrometheusExporter, let 
Prometheus find the port itself
+      links:
+        - name: GitHub Issue
+          url: https://github.com/apache/solr-operator/issues/483
+        - name: GitHub PR
+          url: https://github.com/apache/solr-operator/pull/539
   artifacthub.io/images: |
     - name: solr-operator
       image: apache/solr-operator:v0.7.0-prerelease

Reply via email to