Skip to content

Commit

Permalink
Fix keda-metrics-apiserver port
Browse files Browse the repository at this point in the history
In our 2.11.2 release we retired the old 9022 metrics port, which was
superseded by port 8080. We updated the service with the port name
"metrics", but it looks like we missed updating the ServiceMonitor,
which still had it listed as "http" -- resulting in the prometheus
target not working.

This updates the ServiceMonitor so the port name matches the service,
and the tubes are properly connected and "metrics" will be a valid
prometheus target for keda-metrics-apiserver.

Signed-off-by: John Kyros <[email protected]>
  • Loading branch information
jkyros committed Mar 8, 2024
1 parent 903e520 commit b7cad3d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion resources/keda-olm-operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ spec:
matchLabels:
app.kubernetes.io/name: keda-metrics-apiserver
endpoints:
- port: http
- port: metrics
interval: 60s
namespaceSelector:
any: false
Expand Down

0 comments on commit b7cad3d

Please sign in to comment.