mstefany opened a new pull request, #706:
URL: https://github.com/apache/apisix-helm-chart/pull/706

   Metrics' port & URL is defined 
[here](https://github.com/apache/apisix-helm-chart/blob/f14f58dc7fbe8a963b346ab322a93c662f191b3a/charts/apisix-ingress-controller/templates/apisix-configmap.yaml#L170-L171)
 (port 9091 & URL `/apisix/prometheus/metrics`).
   
   And it doesn't match what's in the 
[serviceMonitor](https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix-ingress-controller/templates/servicemonitor.yaml#L33).
 TargetPort should be `prometheus`, a.k.a. 9091, and path is not specified 
which defaults to `/metrics` which is incorrect.
   
   Fact that it's not working in default configuration is visible in my logs:
   ```
   apisix-ingress-controller-6b4d5cc778-f582v apisix 10.254.82.207 - - 
[17/Jan/2024:11:00:22 +0000] 10.254.83.238:9080 "GET /metrics HTTP/1.1" 404 47 
0.000 "-" "Prometheus/2.48.1" - - - "http://10.254.83.238:9080";
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to