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

kubernetes: automatically determine the cluster CIDRs for reverse lookups #4748

Open
johnbelamaric opened this issue Jul 12, 2021 · 2 comments

Comments

@johnbelamaric
Copy link
Member

What would you like to be added:
Currently we manually configure the cluster CIDR, since it's not available from the API. This looks like it may be changing, see:

kubernetes/kubernetes#46508
kubernetes/enhancements#2593

Why is this needed:

If we can pull the cluster CIDRs from the API, then we can figure out the right ranges for reverse lookups, avoiding fallthrough for the reverse ranges and also avoiding manual reconfiguration when the cluster CIDRs change.

@johnbelamaric johnbelamaric changed the title kubernetes: automatically determine the cluster IPs for reverse lookups kubernetes: automatically determine the cluster CIDRs for reverse lookups Jul 12, 2021
@chrisohaver chrisohaver self-assigned this Jul 12, 2021
@chrisohaver
Copy link
Member

We'd need the Cluster CIDR (for Pod IP range) and also the Service CIDR (for Service IP range).
kubernetes/enhancements#2593 seems to be leaning toward only addressing Cluster CIDR.

I've read through both kubernetes/kubernetes#46508 and kubernetes/enhancements#2593, it's unclear to me if/how these CIDRs are already available in the API. One 3 year old comment says yes (although I can't figure out if/where those APIs are exposed), most others say no. kubernetes/enhancements#2593 implies strongly they are already available (e.g. with reference to Service CIDR being available in "the Service API").

@chrisohaver
Copy link
Member

Keeping tabs on this - I don't see any movement on the KEP (No open PR to implement it yet that I can find).

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

No branches or pull requests

2 participants