Skip to content

Commit 6ab0b8a

Browse files
committed
docs(helm): correct the metrics bindAddress override note
The earlier note suggested keeping `service.port` aligned with a runtime override of `METRICS_BIND_ADDRESS`, but `service.port` is the cluster-facing front-door port and has no effect on the Service backend. The Service `targetPort` and the pod `containerPort` are both derived from `bindAddress` at template time, so overriding the env var via `controller.env` cannot move the rendered Service at all — that operational consequence is what the comment now states, with `bindAddress` flagged as the only chart-side knob that moves the listener and the Service together. Addresses review feedback on kagent-dev#1803. Signed-off-by: Daniel Orbach <ddorbach@gmail.com>
1 parent 2f9c8d0 commit 6ab0b8a

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

helm/kagent/values.yaml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -229,14 +229,14 @@ controller:
229229
# - name: OPENSHELL_GRPC_ADDR
230230
# value: "openshell.my-namespace.svc.cluster.local:8080"
231231
# -- Prometheus-style /metrics endpoint for the controller manager.
232-
# When enabled, provisions a dedicated metrics Service plus the ClusterRoles
233-
# required for authenticated scrapes. Bind `<fullname>-metrics-reader` to
234-
# your Prometheus ServiceAccount to grant scrape access. The Service
235-
# targetPort and pod containerPort are derived from `bindAddress` so the
236-
# two cannot drift; only the cluster-facing `service.port` is independently
237-
# configurable. Overriding `METRICS_BIND_ADDRESS` via `controller.env`
238-
# changes the runtime bind but not the Service — keep `service.port`
239-
# aligned manually if you take that escape hatch.
232+
# When enabled, provisions a dedicated metrics Service plus the
233+
# ClusterRoles required for authenticated scrapes. Bind
234+
# `<fullname>-metrics-reader` to your Prometheus ServiceAccount to grant
235+
# scrape access. Use `bindAddress` for any port change: the Service
236+
# `targetPort` and the pod `containerPort` are derived from it at
237+
# template time, so overriding `METRICS_BIND_ADDRESS` via
238+
# `controller.env` shifts only the runtime listener and leaves the
239+
# rendered Service pointing at the chart-time port.
240240
# @default -- disabled
241241
metrics:
242242
enabled: false

0 commit comments

Comments
 (0)