-
Notifications
You must be signed in to change notification settings - Fork 268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Set proper pod port where prometheus metrics are exposed #869
fix: Set proper pod port where prometheus metrics are exposed #869
Conversation
9ccb70f
to
f8f1dc9
Compare
Signed-off-by: randrusiak <r.andrusiak@protonmail.com>
f8f1dc9
to
20c5ed2
Compare
@@ -321,7 +321,7 @@ metadata: | |||
namespace: kuberhealthy | |||
annotations: | |||
prometheus.io/scrape: "true" | |||
prometheus.io/port: "80" | |||
prometheus.io/port: "8080" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will get regenerated by our build automation when the helm chart is updated - fyi
@@ -10,7 +10,7 @@ metadata: | |||
annotations: | |||
{{- if and .Values.prometheus.enabled (not .Values.prometheus.serviceMonitor.enabled) }} | |||
prometheus.io/scrape: "true" | |||
prometheus.io/port: "80" | |||
prometheus.io/port: "8080" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joshulyne can you verify 8080 is the right port here??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yupp, they're right!
Currently, prometheus-server tries to scrape metrics from pod's port 80 which isn't exposed.
According to default prometheus kubernetes_sd_config service annotation
prometheus.io/port
should point to pod's port instead of service port.