-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Kube-dns should use versioned client-go #31929
Comments
Would like to work on this. I'm using versioned client-go for cluster-proportional-autoscaler too. |
I met some difficulty when tried to use client-go library with current codes. The main issue is that kube-dns is using And in kubernetes/client-go#4, people mentioned that somebody proposed a refactoring to the |
ACK. If we're going to do major surgery to kube-dns we should consider On Mon, Sep 12, 2016 at 2:38 PM, Zihong Zheng notifications@github.com
|
Seems like it is an oversight and folks intended to include this Informer library too. I will again work on this after this library is included. |
Coredns has its first release available (https://github.com/miekg/coredns/releases) might be a good time to check it out. |
Yeah, I'll move informer to the library, too. Perhaps next week. |
@bowei is this done now? |
It currently uses client-go, but not the versioned one -- opened issue kubernetes/dns#35 |
closing as this is tracked in the dns repo |
/assign |
/close |
In prep to maybe move kube-dns to its own repo, we should adopt the new client lib.
The text was updated successfully, but these errors were encountered: