Skip to content

[kube-prometheus-stack] Allow to configure kube-api-server service monitor path #5313

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lucaspouzac
Copy link

For EKS cluster, it's necessary to update path of serviceMonitor to scrape metrics (need to add /metrics).

I add this optional configuration on this serviceMonitor.

Signed-off-by: Lucas POUZAC <lucas.pouzac@lpoconseil.fr>
@lucaspouzac lucaspouzac changed the title [kube-api-server] Configuration of serviceMonitor path [kube-prometheus-stack] Allow to configure kube-api-server service monitor path Feb 12, 2025
@jkroepke
Copy link
Member

I'm a bit confused here, since /metrics is the default value, if not defined.

@lucaspouzac
Copy link
Author

Maybe because I use alloy to scrape service monitor ?

@jkroepke
Copy link
Member

if so, it feels like an bug in alloy.

Copy link

stale bot commented Apr 25, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants