Skip to content
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

Only allow single K8S API server in SD config #2013

Closed
fabxc opened this Issue Sep 21, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@fabxc
Copy link
Member

fabxc commented Sep 21, 2016

The Kubernetes upstream client library does not support multiple API server URLs but instead expects DNS load balancing or using a proxy:

https://godoc.org/k8s.io/client-go/1.4/rest#Config

We should consider making the K8S SD config argument just a single API server URL. If we want to use the official client library there's no way to use multiple ones. With cluster federation and all, there's a fair bit of proxying involved behind the scenes anyway.

@brian-brazil @jimmidyson

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Sep 21, 2016

If that's the way the upstream API works, then that's the way it works.

@jimmidyson

This comment has been minimized.

Copy link
Member

jimmidyson commented Sep 21, 2016

OK let's go with that. Needs to be noted that as we move to static configuration for API servers to scrape metrics from (not k8s SD) that users will have to configure all the API servers, not just the RR DNS or proxy address, but that's a separate issue.

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Oct 26, 2016

The new k8 SD does this.

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.